The default bug view has changed. See this FAQ.

Some css rules in existing css files doesn't apply anymore (bookmarks-ptf gone, structure have changed)

RESOLVED FIXED in seamonkey2.1b1

Status

SeaMonkey
Themes
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: stefanh, Assigned: stefanh)

Tracking

Trunk
seamonkey2.1b1
x86
All

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

7 years ago
http://mxr.mozilla.org/comm-central/search?find=%2Fsuite%2F&string=bookmarks-ptf
(Assignee)

Comment 1

7 years ago
Fixing this will also fix bug 586026, comment #10
(Assignee)

Comment 2

7 years ago
Note also that rules (navigator.css) like "#personal-bookmarks > .bookmarks-toolbar-customize" doesn't work anymore.
(Assignee)

Updated

7 years ago
Summary: id="bookmarks-ptf" doesn't exist anymore, but dependent style rules remains → Some css rules in existing css files doesn't apply anymore (bookmarks-ptf gone, structure have changed)

Comment 3

7 years ago
Some of this was noted in the comments I pointed to in bug 585601 and I had in mind to fix it there, but unfortunately, landing places bookmarks hasn't decreased my TODO list, but made it longer and since then I couldn't come to do much real work, for whatever reasons. I hope to get to this some time soon, but might not be this week.
(Assignee)

Comment 4

7 years ago
Created attachment 465391 [details] [diff] [review]
Fix navigator.css in all themes

I think this is all that needs to be done (assuming this just affected navigator.css).

While fixing this I noted some nits that needs to be fixed in the mac version, bit I'll file a follow-up on that.
Assignee: nobody → stefanh
Status: NEW → ASSIGNED
Attachment #465391 - Flags: superreview?(neil)
Attachment #465391 - Flags: review?(neil)

Comment 5

7 years ago
Comment on attachment 465391 [details] [diff] [review]
Fix navigator.css in all themes

> /* Prevent [mode="icons"|"text"] from hiding the label and icon */
>-#bookmarks-ptf .bookmark-item > .toolbarbutton-text,
>-#bookmarks-ptf .bookmark-item > .toolbarbutton-icon {
>+#PlacesToolbar .bookmark-item > .toolbarbutton-text,
>+#PlacesToolbar .bookmark-item > .toolbarbutton-icon {
>   display: -moz-box !important;
KaiRo added an inferior rule to BookmarksToolbar.css :-(

Comment 6

7 years ago
(In reply to comment #5)
> KaiRo added an inferior rule to BookmarksToolbar.css :-(

Why inferior? Is there any case where .bookmark-item is not inside a #PlacesToolbar and needs to follow those modes?
(Assignee)

Comment 7

7 years ago
Created attachment 466161 [details] [diff] [review]
Now including bookmarksToolbar.css
Attachment #465391 - Attachment is obsolete: true
Attachment #466161 - Flags: superreview?(neil)
Attachment #466161 - Flags: review?(neil)
Attachment #465391 - Flags: superreview?(neil)
Attachment #465391 - Flags: review?(neil)

Comment 8

7 years ago
(In reply to comment #6)
> (In reply to comment #5)
> > KaiRo added an inferior rule to BookmarksToolbar.css :-(
> Why inferior?
You went to all the trouble of making favicons work in bookmarks, and then you let people turn them off ;-)

Comment 9

7 years ago
(In reply to comment #8)
> (In reply to comment #6)
> > (In reply to comment #5)
> > > KaiRo added an inferior rule to BookmarksToolbar.css :-(
> > Why inferior?
> You went to all the trouble of making favicons work in bookmarks, and then you
> let people turn them off ;-)

Well, for one thing, I heard people talk of not being able to turn them off as a bug, but OK, I see that side - the rule itself without the space selector is superior than the unneededly longer rule in bookmarks.css though :P
Comment on attachment 466161 [details] [diff] [review]
Now including bookmarksToolbar.css

>-toolbarpaletteitem[place] > #personal-bookmarks > .bookmarks-toolbar-customize {
>+toolbarpaletteitem[place] > #personal-bookmarks > #PlacesToolbar > .bookmarks-toolbar-customize {
Eww, this is ugly. KaiRo, why do we need a separate #PlacesToolbar ?
(In reply to comment #9)
> Well, for one thing, I heard people talk of not being able to turn them off as
> a bug, but OK, I see that side - the rule itself without the space selector is
> superior than the unneededly longer rule in bookmarks.css though :P
Possibly the rule shouldn't apply to the Bookmarks button itself.
Comment on attachment 466161 [details] [diff] [review]
Now including bookmarksToolbar.css

> .bookmarks-toolbar-customize {
...
>   display: none;
...
> }
> 
>+toolbarpaletteitem[place] > #personal-bookmarks > #PlacesToolbar > .bookmarks-toolbar-customize {
>   display: -moz-box;
> }
...
>+toolbarpaletteitem[place="toolbar"] > #personal-bookmarks > #PlacesToolbar > .bookmarks-toolbar-overflow-items,
>+toolbarpaletteitem[place="toolbar"] > #personal-bookmarks > #PlacesToolbar > .bookmarks-toolbar-items {
>   visibility: collapse;
> }
> 
>+toolbarpaletteitem[place="palette"] > #personal-bookmarks > #PlacesToolbar > .bookmarks-toolbar-overflow-items,
>+toolbarpaletteitem[place="palette"] > #personal-bookmarks > #PlacesToolbar > .bookmarks-toolbar-items {
>   display: none;
> }
These visibility/display rules (and their equivalents in other themes) now have equivalents in content places.css and aren't needed here any more.
Attachment #466161 - Flags: superreview?(neil)
Attachment #466161 - Flags: review?(neil)
Attachment #466161 - Flags: review-
(Assignee)

Comment 13

7 years ago
Created attachment 469588 [details] [diff] [review]
Now without removed rules

Right, these rules are now gone - turns out that the patch was bitrottened.
Attachment #466161 - Attachment is obsolete: true
Attachment #469588 - Flags: superreview?(neil)
Attachment #469588 - Flags: review?(neil)
Comment on attachment 469588 [details] [diff] [review]
Now without removed rules

[Note to self: Bookmarks items don't display correctly in text mode, with or without the patch, so must be a separate bug. Sigh.]
Attachment #469588 - Flags: superreview?(neil)
Attachment #469588 - Flags: superreview+
Attachment #469588 - Flags: review?(neil)
Attachment #469588 - Flags: review+
(Assignee)

Comment 15

7 years ago
http://hg.mozilla.org/comm-central/rev/1332ded22012
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b1
You need to log in before you can comment on or make changes to this bug.