Closed Bug 848455 Opened 7 years ago Closed 7 years ago

Comment anchor links do not scroll to the correct part of the page in the Mozilla theme

Categories

(bugzilla.mozilla.org Graveyard :: Sandstone/Mozilla Skin, defect)

Production
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ehsan, Assigned: KWierso)

References

Details

Attachments

(1 file, 2 obsolete files)

Component: User Interface → Sandstone
OS: Mac OS X → All
Hardware: x86 → All
Not a regression as the Sandstone skin is new for BMO 4.2.
No longer blocks: bmo_4_2_regressions
This seems to work for me. With this patch applied, clicking a link to a comment shows that comment's header in full below the fixed site header.

This invisible margin's height is hardcoded to 80px, which seems to work for me. Can the fixed site header be taller/shorter than ~80px? Is there a better way to adjust to the header's height than this?
Assignee: nobody → kwierso
Status: NEW → ASSIGNED
Attachment #736187 - Flags: feedback?
(In reply to Wes Kocher (:KWierso) from comment #2)
> Is there a better way to adjust to the header's height than this?

Also, should this apply to more than the bug comment anchors?
I must've uploaded an incomplete version of the patch last time. Fixed in this version, hopefully.
Attachment #736187 - Attachment is obsolete: true
Attachment #736187 - Flags: feedback?
Attachment #737201 - Flags: feedback?(dkl)
Bram, since you are heavily into this part of BMO at the moment, can you take a look at Wes' changes and see if they look good and we can get the fixes committed?

Thanks
dkl
Flags: needinfo?(bram)
kwierso’s CSS change looks good to me.

The only suggestion I would make is to shift the margin-top and height values by 15px (so they become -95px and 95px, respectively). This way, each comment gets a white space between it and the header.

Other than this, we’re good to go for committing the fix
Flags: needinfo?(bram)
Duplicate of this bug: 894989
Updated version with 95/-95 values.

I don't think I have commit access to BMO, so someone else would have to push it.
Attachment #737201 - Attachment is obsolete: true
Attachment #737201 - Flags: feedback?(dkl)
Flags: needinfo?(bram)
I will check it in.
Flags: needinfo?(bram)
Committing to: bzr+ssh://dlawrence%40mozilla.com@bzr.mozilla.org/bmo/4.2
modified skins/contrib/Mozilla/global.css
Committed revision 8883.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Depends on: 898999
There are quite a few other places that use named anchors that are affected by this and have the same issue as the comments of show_bug.cgi.

Glob and I discussed this and we will remove the fixed position setting for the header for now and address this in a different bug to make sure we find a solution that will work everywhere. We would like to get this skin as the default soon so we feel it is a good tradeoff for now.

Committing to: bzr+ssh://dlawrence%40mozilla.com@bzr.mozilla.org/bmo/4.2
modified skins/contrib/Mozilla/global.css
Committed revision 8948.

dkl
Blocks: 905763
No longer blocks: 905763
Product: bugzilla.mozilla.org → bugzilla.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.