Library infoPane does not shrink if localstore has an height set up

VERIFIED FIXED in Firefox 3

Status

()

Firefox
Bookmarks & History
VERIFIED FIXED
10 years ago
9 years ago

People

(Reporter: mak, Assigned: mak)

Tracking

Trunk
Firefox 3
Points:
---
Bug Flags:
blocking-firefox3 -
wanted-firefox3 +
in-litmus +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

10 years ago
After landing of bug 403140 the infoPane in the right of the Library is no more resizable, but the old height could still be set up in the localstore.rdf for
it. It should shrink when nothing is selected to leave some more space, but if
the height is set up there it does not shrink.

Drivers: should we change the infoPane id so height is discarded, or annotate this for beta users (suggesting to delete localstore)?
Flags: blocking-firefox3?
What's the impact here? What do you mean by annotate it? Relnote? Did we set up the height in localstore?
(Assignee)

Comment 2

10 years ago
no, for annotate i mean adding this into the KB page that QA is preparing for beta users, bug 426340.

impact is that you loose some precious space from the results tree, and have white space under options in the infoPane. the worst case is if you had closed Library with the details pane open (more clicked) or resized to be bigger, then you upgrade, most probably half of Library right pane is lost in this case and you can't resize it anymore.

We persisted height before, so we were saving it in localstore
Created attachment 314722 [details]
Not auto resized details pane

Like Marco said, if the height is persistent within the localstore.rdf the details pane isn't auto sized.
(Assignee)

Comment 4

10 years ago
Created attachment 314830 [details] [diff] [review]
patch

change the id from infoPane to infoPaneBox
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Attachment #314830 - Flags: review?(mano)
Not a blocker, but good cleanup if we get a reviewed patch.
Flags: wanted-firefox3+
Flags: blocking-firefox3? → blocking-firefox3-
Comment on attachment 314830 [details] [diff] [review]
patch

places.js calss getElementById("infoBox"), and there are few more references in pinstripe.
Attachment #314830 - Flags: review?(mano) → review-
(Assignee)

Comment 7

10 years ago
(In reply to comment #6)
> (From update of attachment 314830 [details] [diff] [review])
> places.js calss getElementById("infoBox"), and there are few more references in
> pinstripe.
> 

i'm not touching infoBox, i'm changing infoPane...
(Assignee)

Updated

10 years ago
Attachment #314830 - Flags: review- → review?(mano)
Comment on attachment 314830 [details] [diff] [review]
patch

ugh, sorry, r=mano.
Attachment #314830 - Flags: review?(mano) → review+
(Assignee)

Comment 9

10 years ago
Comment on attachment 314830 [details] [diff] [review]
patch

asking approval to fix a bad behaviour for beta users (infopane steal space in the Lbrary, see screenshot above)
Attachment #314830 - Flags: approval1.9?
Comment on attachment 314830 [details] [diff] [review]
patch

a1.9=beltzner
Attachment #314830 - Flags: approval1.9? → approval1.9+
(Assignee)

Updated

10 years ago
Keywords: checkin-needed
mozilla/browser/components/places/content/places.js 	1.155
mozilla/browser/components/places/content/places.xul 	1.130
mozilla/browser/themes/gnomestripe/browser/places/organizer.css 	1.15
mozilla/browser/themes/pinstripe/browser/places/organizer.css 	1.12
mozilla/browser/themes/winstripe/browser/places/organizer.css 	1.11
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3
Verified with the following builds. Switching through the bookmarks, folders and tags automatically shrinks the details pane now for existing profiles.

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9pre) Gecko/2008041506 Minefield/3.0pre

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.4; en-US; rv:1.9pre) Gecko/2008041504 Minefield/3.0pre ID:2008041504
Status: RESOLVED → VERIFIED
Flags: in-litmus?
Test cases were created on 3.0 and 3.1 test runs on litmus for regression testing.

For 3.0,
https://litmus.mozilla.org/show_test.cgi?id=7533

For 3.1,
https://litmus.mozilla.org/show_test.cgi?id=7534
Flags: in-litmus? → in-litmus+
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.