Bookmarks organizer splitters should be 1 pixel wide on Vista

VERIFIED FIXED in Firefox 3

Status

()

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

People

(Reporter: faaborg, Unassigned)

Tracking

({polish})

Trunk
Firefox 3
x86
Windows Vista
polish
Points:
---
Dependency tree / graph
Bug Flags:
blocking-firefox3 -

Firefox Tracking Flags

(Not tracked)

Details

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

(Reporter)

Description

11 years ago
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?

Updated

11 years ago
Flags: blocking-firefox3? → blocking-firefox3-
Whiteboard: [vistatheme]

Comment 1

11 years ago
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

11 years ago
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.

Comment 4

11 years ago
The vertical resizer does have more than 1 pixel, but the horizontal one is only one pixel high.
(Reporter)

Updated

11 years ago
Blocks: 423491
(Reporter)

Comment 5

10 years ago
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
(Reporter)

Updated

10 years ago
Blocks: 425582
(Reporter)

Updated

10 years ago
Blocks: 405605

Updated

10 years ago
Whiteboard: [vistatheme] → [vistatheme][possible patch in bug 403147]

Comment 6

10 years ago
see bug 403147 for a possible impl

Comment 7

10 years ago
FIXED by checkin for bug 403147
Status: NEW → RESOLVED
Last Resolved: 10 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.