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
libphutilfc84916a1a31865f8b48d99d98766b2a1c29ef3a
arcanist10e5194752901959507223c01e0878e6b8312cc5
phabricatorc8281bc339277290ce76c5679881c1da8f1a2df2

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.

Confirmed this is fixed after upgrading to 864d3984ebdb42b9a2670103ca3607342e46b7dc

(Upgraded from 5efbf4d74aa72c5a7d7f161ea8c5aa33ac0e3189)