Closed
Bug 720975
Opened 11 years ago
Closed 11 years ago
Switch to vertical layout only when the window width < 550px
Categories
(DevTools :: Style Editor, defect)
Tracking
(firefox11 verified)
RESOLVED
FIXED
Firefox 12
Tracking | Status | |
---|---|---|
firefox11 | --- | verified |
People
(Reporter: paul, Assigned: paul)
Details
(Whiteboard: [qa+][testday-20120203][qa!:11])
Attachments
(2 files)
6.44 KB,
patch
|
msucan
:
review+
cedricv
:
feedback+
akeybl
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
6.51 KB,
patch
|
Details | Diff | Splinter Review |
To avoid some frustration when the window is large enough to get the horizontal view.
Assignee | ||
Comment 1•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Attachment #591456 -
Flags: review?(cedricv)
Assignee | ||
Updated•11 years ago
|
Attachment #591456 -
Flags: review?(cedricv) → feedback?(cedricv)
Assignee | ||
Updated•11 years ago
|
Whiteboard: [addToFirefox11]
Assignee | ||
Updated•11 years ago
|
Attachment #591456 -
Flags: review?(mihai.sucan)
Comment 2•11 years ago
|
||
Comment on attachment 591456 [details] [diff] [review] patch v1 Review of attachment 591456 [details] [diff] [review]: ----------------------------------------------------------------- Patch looks good. Thanks Paul!
Attachment #591456 -
Flags: review?(mihai.sucan) → review+
Updated•11 years ago
|
Attachment #591456 -
Flags: feedback?(cedricv) → feedback+
Assignee | ||
Updated•11 years ago
|
Whiteboard: [addToFirefox11] → [land-in-fx-team][addToFirefox11]
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → paul
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•11 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/f0f0451c75eb
Whiteboard: [land-in-fx-team][addToFirefox11] → [fixed-in-fx-team][addToFirefox11]
Comment 4•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/f0f0451c75eb
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team][addToFirefox11] → [addToFirefox11]
Target Milestone: --- → Firefox 12
Assignee | ||
Comment 5•11 years ago
|
||
Comment on attachment 591456 [details] [diff] [review] patch v1 [Approval Request Comment] Regression caused by (bug #): Not a regression, new feature. User impact if declined: Bad UX. The StyleEditor UI will constantly be reorganized on resize. Testing completed (on m-c, etc.): on m-c. Risk to taking this patch (and alternatives if risky): low. Only CSS.
Attachment #591456 -
Flags: approval-mozilla-aurora?
Comment 6•11 years ago
|
||
Comment on attachment 591456 [details] [diff] [review] patch v1 [Triage Comment] Low risk fix in support of a new feature. Approved for Aurora.
Attachment #591456 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 7•11 years ago
|
||
Assignee | ||
Comment 8•11 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/e1d6266b5a2c
status-firefox11:
--- → fixed
Whiteboard: [addToFirefox11]
![]() |
||
Comment 9•10 years ago
|
||
Seems to work on Linux-x86_64 with * 11.0b1 as opposed to Nightly 20120125. * Aurora Gecko/20120202 Firefox/12.0a2 as opposed to Gecko/20120101 Firefox/11.0a2 However, on my system it means that the file list can be much larger than the editor while there is a lot of blank space. If the text was larger, even less of it would fit, and/or more horizontal scrolling would appear. Do the text sizes depend on anything, and if yes, what about deriving the width from them? See also bug 719457.
Whiteboard: [qa+] → [qa+][testday-20120203]
Assignee | ||
Comment 10•10 years ago
|
||
(In reply to Aleksej [:Aleksej] from comment #9) > Seems to work on Linux-x86_64 with > * 11.0b1 as opposed to Nightly 20120125. > * Aurora Gecko/20120202 Firefox/12.0a2 as opposed to Gecko/20120101 > Firefox/11.0a2 > > However, on my system it means that the file list can be much larger than > the editor while there is a lot of blank space. "blank space"? > If the text was larger, Which text? > even less of it would fit, and/or more horizontal scrolling would appear. Do the > text sizes depend on anything, and if yes, what about deriving the width > from them? See also bug 719457. Sorry - I am not sure to understand. Can you open a new bug with screenshots?
![]() |
||
Comment 11•10 years ago
|
||
(In reply to Paul Rouget [:paul] from comment #10) https://bugzilla.mozilla.org/show_bug.cgi?id=723909 https://bugzilla.mozilla.org/show_bug.cgi?id=723912
Comment 12•10 years ago
|
||
Mozilla/5.0 (Windows NT 6.1; rv:11.0) Gecko/20100101 Firefox/11.0 Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:11.0) Gecko/20100101 Firefox/11.0 Mozilla/5.0 (X11; Linux x86_64; rv:11.0) Gecko/20100101 Firefox/11.0 Verified using Firefox 11 beta 1 on Win 7, Ubuntu 11.10 and Mac OS X 10.6 that the Style Editor switches to vertical only when the width is < 550 px.
Whiteboard: [qa+][testday-20120203] → [qa+][testday-20120203][qa!:11]
Updated•4 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•