Closed Bug 403138 Opened 12 years ago Closed 12 years ago

Bookmarks organizer splitters should be 1 pixel wide on Vista

Categories

(Firefox :: Bookmarks & History, defect)

x86
Windows Vista
defect
Not set

Tracking

()

VERIFIED FIXED
Firefox 3

People

(Reporter: faaborg, Unassigned)

References

Details

(Keywords: polish, Whiteboard: [vistatheme][possible patch in bug 403147])

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.
Flags: blocking-firefox3?
Flags: blocking-firefox3? → blocking-firefox3-
Whiteboard: [vistatheme]
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;
}
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.
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.
The vertical resizer does have more than 1 pixel, but the horizontal one is only one pixel high.
Blocks: 423491
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
Blocks: 425582
Blocks: 405605
Whiteboard: [vistatheme] → [vistatheme][possible patch in bug 403147]
see bug 403147 for a possible impl
FIXED by checkin for bug 403147
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3
Verified with Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9pre) Gecko/2008050806 Minefield/3.0pre ID:2008050806
Status: RESOLVED → VERIFIED
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.