403138 - Bookmarks organizer splitters should be 1 pixel wide on Vista

Status: VERIFIED FIXED

(Firefox :: Bookmarks & History, defect)

Description

[Alex Faaborg [:faaborg] (Firefox UX)]

The splitter between the tree view and list view in the bookmarks organizer, and the splitter between the preview area and the list view should both be 1 pixel wide on Vista.

See bug# 393514 for a mockup.

Comment 1

[Lim Chee Aun]

Here's a quick fix, using userChrome.css:

#placesView #splitter{
	border: 0 !important;
	-moz-border-right-colors: none !important;
	border-right: 1px solid #C1D1E3 !important;
	background-color: transparent !important;
	min-width: 0 !important;
}

#placesView #contentSplitter{
	border: 0 !important;
	-moz-border-bottom-colors: none !important;
	border-bottom: 1px solid #C1D1E3 !important;
	background-color: transparent !important;
	min-height: 0 !important;
}

#placesView #splitter>grippy,
#placesView #contentSplitter>grippy{
	display: none !important;
}

Comment 2

[Victor Bielawski]

One pixel? That would be terribly hard to resize. I propose a width of 3 pixels with the middle pixel grey and the outer pixels white.

Comment 3

[Luke Iliffe (Harlequin99)]

I have just tried the uc code suggested in comment #1 and it is easy to resize. The area in which the mouse pointer turns double headed seems to be wider than the elegant 1px wide border itself. This is on a 1680 X 1050 LCD display.

Comment 4

[Victor Bielawski]

The vertical resizer does have more than 1 pixel, but the horizontal one is only one pixel high.

Comment 5

[Alex Faaborg [:faaborg] (Firefox UX)]

Note that the color of these splitters should be #a9b7c9 (from windows media player) hard coded system colors need to be defined in bug 425598

Comment 6

[Marco Bonardo [:mak]]

see bug 403147 for a possible impl

Comment 7

[Ryan A. C.]

FIXED by checkin for bug 403147

Comment 8

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Verified with Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9pre) Gecko/2008050806 Minefield/3.0pre ID:2008050806

Comment 9

[Gervase Markham [:gerv]]

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