Page MenuHomePhabricator

Differential comment panel haunter does not allow scrolling to bottom of differential revision when no ui.footer-items configured
Closed, ResolvedPublic

Description

Using the Z shortcut to show the comment panel haunter causes the end of the revision to not be totally visible. This is more noticeable on larger revisions. When using the first haunt mode (single Z), the height seems to be off by ~15px, as the cutoff seems to happen midway between the last line in diff (in the screenshots, line 893 is the last line of the displayed context).

Screen Shot 2016-09-19 at 9.17.43 AM.png (842×1 px, 81 KB)

Using the second haunt mode Z+Z the height seems to be off by the height of the comment panel itself.

Screen Shot 2016-09-19 at 9.28.26 AM.png (589×1 px, 56 KB)

I am unable to reproduce this on this install (secure.phabricator.com) using the same browsers/systems - the height behavior here seems to work properly. I've tried setting up a new instance on Phacility however was unable to get a test repository working to where I could push to it. I will continue to try and get this setup on a test phacility instance.

I've seen this for several weeks now (since I learned of comment haunter). I recently upgraded over the weekend and verified that I'm still seeing this issue.

Environment

I can check on Windows systems if need be

BrowserOS
Firefox 48.0.1OSX 10.10.5, 10.11.5
Safari 9.1.3OSX 10.10.5, 10.11.5
Chrome 53OSX 10.10.5, 10.11.5

The following are the configs I would think might affect this behavior, but the majority of them are using the default.

ConfigValue
ui.header-color"green"
ui.logoNone
ui.footer-itemsNone
User ConfigValue
Show Unified DiffsDefault (On Small Screens)
Show Older InlinesDefault (Enabled)
Show FiletreeDefault (Enable Filetree)
AccessibilityUse High Contrast Colors

Event Timeline

Ok I have observed this now on this install, however only in the second haunt mode Z+Z:
https://secure.phabricator.com/D15067

I played around a bit, it looks like adding a footer helps to match the behavior on this install. When I configured ui.footer-items the first haunt mode would make all code visible (though the footer was not), and I think that's why it doesn't appear to be an issue here.

cspeckmim renamed this task from Differential comment panel haunter does not allow scrolling to bottom of differential revision to Differential comment panel haunter does not allow scrolling to bottom of differential revision when no ui.footer-items configured.Sep 20 2016, 4:04 AM
epriestley claimed this task.
epriestley added a subscriber: epriestley.

This should be resolved by T12049 + T11114, once Differential moves to the new comment area UI. The new universal haunted mode puts a spacer into the document while the panel is haunted, so if the panel is 500px tall we put a 500px empty space at the bottom of the document. This means that all content is always visible while scrolling.

To cut down on bookkeeping I'm going to mark this as resolved now -- it won't actually be fixed in any version of the codebase that is available to anyone until D17106 lands, though. Feel free to file a followup if you run into issues after that.