964413 - Don't hide the selected anchor behind the search panel

Status: RESOLVED FIXED

(Webtools Graveyard :: DXR, defect, P2)

Description

[Erik Rose [:erik][:erikrose]]

When you go to, for example, http://dxr.allizom.org/mozilla-central/source/accessible/public/nsIAccessibilityService.h#26, line 26 gets hidden behind the absolutely positioned search panel. Let's make it not do that. Nick's mentioned he'd like it in the center of the viewport, which is what the production JS does now. I could also live with a few lines below the search panel. The common theme here is to show some context on each side.

The old implementation used an onhashchange piece of JS to do this. It might also be possible with clever use of padding in plain CSS. https://stackoverflow.com/questions/10732690/offsetting-an-html-anchor-to-adjust-for-fixed-header

If we end up redoing the markup for performance (see bug 964091), it would probably make sense to do that first.

Comment 1

[Schalk Neethling [:espressive]]

Ah yes, I have seen this and was meaning to fix it. Thanks for filing the bug, will sort it out.

Comment 2

[Schalk Neethling [:espressive]]

(In reply to Erik Rose [:erik][:erikrose] from comment #0)
> When you go to, for example,
> http://dxr.allizom.org/mozilla-central/source/accessible/public/
> nsIAccessibilityService.h#26, line 26 gets hidden behind the absolutely
> positioned search panel.....

I believe in our scenario a little bit of JS is what is going to be required. The CSS only solution definitely assumes a different source structure than what we have.

Comment 3

[[github robot]]

Commit pushed to ui at https://github.com/mozilla/dxr

https://github.com/mozilla/dxr/commit/1296b6a2629da7a29022ebf73d2b637934282c34
Fix Bug 964413, ensure that highlighted anchor is never hidden behind header