Closed
Bug 329761
Opened 18 years ago
Closed 18 years ago
Bookmark properties window is rather tall
Categories
(Firefox :: Bookmarks & History, defect, P3)
Firefox
Bookmarks & History
Tracking
()
RESOLVED
FIXED
Firefox 2 alpha1
People
(Reporter: tuukka.tolvanen, Assigned: tuukka.tolvanen)
References
Details
(Keywords: fixed1.8.1)
Attachments
(1 file)
935 bytes,
patch
|
mozilla
:
review+
bugs
:
superreview+
|
Details | Diff | Splinter Review |
Bookmark properties is >3x the height of my screen and, after resizing, leaves bottom buttons and part of the folder tree box cropped. (I have a bunch of crufty old bookmark folders from, uh, the 90s or something)
Assignee | ||
Comment 1•18 years ago
|
||
Assignee: nobody → tuukka.tolvanen
Status: NEW → ASSIGNED
Attachment #214422 -
Flags: superreview?(bugs)
Attachment #214422 -
Flags: review?(joe)
Assignee | ||
Updated•18 years ago
|
Attachment #214422 -
Attachment description: p1: → p1: an empirical sprinkling of flex
Assignee | ||
Comment 2•18 years ago
|
||
*** Bug 329606 has been marked as a duplicate of this bug. ***
Assignee | ||
Updated•18 years ago
|
OS: Linux → All
Hardware: PC → All
Version: unspecified → Trunk
Comment 3•18 years ago
|
||
Does this also fix Bug 329607 ?
Assignee | ||
Comment 4•18 years ago
|
||
no, I don't see any evidence of size persistence with this patch
Comment 5•18 years ago
|
||
Ria: No, size isn't currently persisted. I've assigned 329607 to myself.
Comment 6•18 years ago
|
||
Comment on attachment 214422 [details] [diff] [review] p1: an empirical sprinkling of flex Though it does fix the upper-bound size problem, the patch as shown creates extra space in the dialog when the user has very few bookmark folders.
Attachment #214422 -
Flags: review?(joe) → review-
Updated•18 years ago
|
Attachment #214422 -
Flags: review- → review+
Comment 7•18 years ago
|
||
Changing my mind to approve the patch now; this tag-style display is going away, but the patch provides temporary relief for some nightly users until that replacement occurs.
Comment 8•18 years ago
|
||
Comment on attachment 214422 [details] [diff] [review] p1: an empirical sprinkling of flex sr=ben@mozilla.org, given joe's caveat. Joe, can you land this?
Attachment #214422 -
Flags: superreview?(bugs) → superreview+
Updated•18 years ago
|
Priority: -- → P3
Target Milestone: --- → Firefox 2 alpha1
Comment 9•18 years ago
|
||
Landed on branch and trunk. Thanks Tuukka!
Comment 10•15 years ago
|
||
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h". In Thunderbird 3.0b, you do that as follows: Tools | Message Filters Make sure the correct account is selected. Click "New" Conditions: Body contains places-to-b-and-h Change the action to "Delete Message". Select "Manually Run" from the dropdown at the top. Click OK. Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter. Gerv
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in
before you can comment on or make changes to this bug.
Description
•