Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 483940

Summary: Display advanced search option on a single line (and move checks to be in front of text)
Product: [ECD] Orion Reporter: Steve Northover <steve_northover>
Component: ClientAssignee: Steve Northover <steve_northover>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: libingw, Silenio_Quarti
Version: 10.0   
Target Milestone: ---   
Hardware: PC   
OS: Windows 7   
See Also: https://git.eclipse.org/r/62263
https://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=11b204bd68fd3af404a4331ee0a7bf4731a6ac91
Whiteboard:
Attachments:
Description Flags
This patch file should be better than a patch in the comment none

Description Steve Northover CLA 2015-12-08 13:01:40 EST
The advanced search boxes ("Case sensitive", "Whole Word", "Regular Expression") are displayed in a column taking up space that could be used as search results.  Let's display them in a row.
Comment 1 Steve Northover CLA 2015-12-08 13:47:08 EST
diff --git a/bundles/org.eclipse.orion.client.ui/web/orion/search/InlineSearchPane.html b/bundles/org.eclipse.orion.client.ui/web/orion/search/InlineSearchPane.html
index 394ba29..0c3b9e7 100644
--- a/bundles/org.eclipse.orion.client.ui/web/orion/search/InlineSearchPane.html
+++ b/bundles/org.eclipse.orion.client.ui/web/orion/search/InlineSearchPane.html
@@ -8,18 +8,18 @@
 	</div>
 	<div class="searchParamBlockDiv">
 		<div class="searchOptionBlock">
-			<div class="searchOption">
-				<label id="advSearchCaseSensitiveLabel" class="searchCheckboxLabel" for="advSearchCaseSensitive"></label>
+			<span class="searchOption">
 				<input type="checkbox" id="advSearchCaseSensitive" class="searchCheckbox">
-			</div>
-			<div class="searchOption">
-				<label id="advSearchWholeWordLabel" class="searchCheckboxLabel" for="advSearchWholeWord"></label>
+				<label id="advSearchCaseSensitiveLabel" class="searchCheckboxLabel" for="advSearchCaseSensitive"></label>
+			</span>
+			<span class="searchOption">
 				<input type="checkbox" id="advSearchWholeWord" class="searchCheckbox">
-			</div>
-			<div class="searchOption">
-				<label id="advSearchRegExLabel" class="searchCheckboxLabel" for="advSearchRegEx"></label>
+				<label id="advSearchWholeWordLabel" class="searchCheckboxLabel" for="advSearchWholeWord"></label>
+			</span>
+			<span class="searchOption">
 				<input type="checkbox" id="advSearchRegEx" class="searchCheckbox">
-			</div>
+				<label id="advSearchRegExLabel" class="searchCheckboxLabel" for="advSearchRegEx"></label>
+			</span>
 		</div>
 		<div class="searchOptionBlock">
 			<div class="searchOption">
Comment 2 Steve Northover CLA 2015-12-08 13:52:55 EST
Created attachment 258517 [details]
This patch file should be better than a patch in the comment
Comment 3 libing wang CLA 2015-12-08 13:58:52 EST
(In reply to Steve Northover from comment #2)
> Created attachment 258517 [details]
> This patch file should be better than a patch in the comment

I am using self hosting. Can't apply the patch from within Orion. Strange.
Comment 4 libing wang CLA 2015-12-08 14:12:01 EST
The patch looks good to me.
Comment 5 Eclipse Genie CLA 2015-12-08 14:31:09 EST
New Gerrit change created: https://git.eclipse.org/r/62263
Comment 7 libing wang CLA 2015-12-08 14:43:38 EST
pushed to master.