Closed Bug 25412 Opened 21 years ago Closed 21 years ago

Search button should be depressable

Categories

(SeaMonkey :: General, defect, P3)

x86
Other
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: michaell, Assigned: bugs)

Details

(Keywords: polish, Whiteboard: [PDT+]w/b minus on 02/25)

The search button should look like a button.  It should have a mouseover and a
depressed state to it.
adding polish keyword
Keywords: polish
reassign the bug to german who will work on this and pass it to don's team. cc 
don.
Assignee: shuang → german
Target Milestone: M14
that's right, also a hover state should be added. Don, while somebody in your 
team is working on navigator.css, can you please add to the current code:
#search-button { 
	list-style-image:none;
	margin: 0px;
	margin-right: 2px;
	background-color: #99CCCC;
	border: 2px solid #99CCCC;
	-moz-border-radius: 4px;
} 
the following two states:

#search-button:hover { 
	background-color: #ccffff;
     padding: 1px;
	border: 1px outset #ccffff;
} 

#search-button:active { 
	background-color: #ccffff;
     padding-top: 2px;
	padding-left: 2px;
	padding-bottom: 0px;
	padding-bottom: 0px;
	border: 1px inset #ccffff;
}

Also the label Search should be all caps like so: SEARCH (change in 
navigator.dtd)

Thanks for pointing this out! 
Assignee: german → don
Moving all UE/UI bugs to new component: User Interface: Design Feedback
UE/UI component will be deleted.
Component: UE/UI → User Interface: Design Feedback
nominating for beta1, this needs to be in before public consumption
Keywords: beta1
PDT+ with the belief that this is really easy and safe.
Whiteboard: [PDT+]
Assignee: don → ben
I agree that it should have mouseover and dpressed behavior, but none of the
other toolbar buttons (i.e. back, forward, etc.) are "standard this way.  I'll
leave it to Ben to duke it out with German on this one. :-)
Must get in fix by 02/23 please.
Whiteboard: [PDT+] → [PDT+]Must fix by 02/23
Whiteboard: [PDT+]Must fix by 02/23 → [PDT+]w/b minus on 02/25
fixed. 
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Claudius, this is probably more your area than mine. If you're swamped, feel
free to push it back to me and I'll gladly do it.
QA Contact: elig → claudius
Assuming this is an XP thing:
I'm reopening for two reasons
1. I don't see the hover state that the above code describes.
2. Where's the border for active and inactive states?

Without these two things the button just shows the bonk (depressed) effect but it's kinda sad looking and does not match any of 
the other UI.
I'm looking at the 2000022509 linux build btw.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
a hover effect for this type of button has not been described anywhere. 
Assignee: ben → german
Status: REOPENED → NEW
Hover should be as described above: #search-button:hover { background-color: #ccffff; padding: 1px; border: 1px outset #ccffff;
Assignee: german → ben
thanks.. will check in. 
Status: NEW → ASSIGNED
don't forget the borders as well.
fix checked in...
Status: ASSIGNED → RESOLVED
Closed: 21 years ago21 years ago
Resolution: --- → FIXED
cc'ing german. German, could you take a quick look at this and verify that your changes were implemented just as you wanted?
marking verified
Status: RESOLVED → VERIFIED
Component: User Interface Design → Browser-General
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.