Closed
Bug 836867
Opened 12 years ago
Closed 12 years ago
The Find Toolbar should transition when opening and closing
Categories
(Toolkit :: Find Toolbar, defect)
Toolkit
Find Toolbar
Tracking
()
RESOLVED
FIXED
mozilla21
People
(Reporter: jaws, Assigned: jaws)
References
(Depends on 1 open bug)
Details
Attachments
(1 file, 2 obsolete files)
3.05 KB,
patch
|
dao
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Attachment #708713 -
Flags: review?(dao)
Comment 1•12 years ago
|
||
Comment on attachment 708713 [details] [diff] [review]
Patch
>+ <property name="hidden">
>+ <getter>
>+ return !this.classList.contains("slideIn") ||
>+ this.hasAttribute("hidden");
>+ </getter>
>+ <setter><![CDATA[
>+ if (val) {
>+ this.addEventListener("transitionend", function onTransitionEnd() {
>+ this.removeEventListener("transitionend", onTransitionEnd);
>+ this.setAttribute("hidden", "true");
>+ })
>+ this.classList.remove("slideIn");
We'd infinitely wait for the transitionend event here when a theme doesn't add a transition. We could set a stub transition in xul.css to handle that.
Attachment #708713 -
Flags: review?(dao) → review-
Assignee | ||
Comment 2•12 years ago
|
||
Attachment #708713 -
Attachment is obsolete: true
Attachment #708747 -
Flags: review?(dao)
This transition will make the bar appear from the bottom on opening and the other way on closing ?
If so the transition will have to be changed when the find bar will be resdesigned (bug 776708).
Comment 4•12 years ago
|
||
Can the .slideIn be changed to .slideOut?
The current patch doesn't allow backwards compatibility for themes, as the
margin-bottom:-1em and opacity:0 cannot be set in FF's before this patch.
By using a slideOut to specify the moving direction one can leave the normal
style rule for findbar as-is:
findbar.slideOut{
margin-bottom:-1em;
opacity:0}
Assignee | ||
Comment 5•12 years ago
|
||
This approach is simpler and should work without causing issues with 3rd-party themes.
Attachment #708747 -
Attachment is obsolete: true
Attachment #708747 -
Flags: review?(dao)
Attachment #710764 -
Flags: review?(dao)
Comment 6•12 years ago
|
||
Comment on attachment 710764 [details] [diff] [review]
Patch v2
> findbar {
> border-top: 2px solid;
> -moz-border-top-colors: ThreeDShadow ThreeDHighlight;
> padding-bottom: 1px;
> min-width: 1px;
>+ transition-property: margin-bottom, opacity, visibility;
>+ transition-duration: 150ms, 150ms, 0s;
>+ transition-timing-function: ease-in-out, ease-in-out, linear;
>+ margin-bottom: 0;
>+ opacity: 1;
>+}
margin-bottom:0 and opacity:1 are the default states and redundant. r=me with them removed
Attachment #710764 -
Flags: review?(dao) → review+
Assignee | ||
Comment 7•12 years ago
|
||
Thanks for the review. I removed the default states and pushed to inbound:
https://hg.mozilla.org/integration/mozilla-inbound/rev/299893af8938
Comment 8•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
You need to log in
before you can comment on or make changes to this bug.
Description
•