mirror of
https://we.phorge.it/source/phorge.git
synced 2024-11-13 02:12:41 +01:00
0f9f8b5f30
Summary: We have some complaints on this feature: - It's not clear that the displayed code is context from gap. - It would be better if the context would be displayed with its real indentation. - It's not clear how far the context is from the displayed code. - Links revealing gap aren't on consistent place. This solves all these problems and introduces new one: It now seems that the reveal links works only with the left side. Anyway, I think that this is better overall. I don't want to put the context on a separate line to not waste space. Test Plan: Displayed various contexts, revealed context. Reviewers: epriestley Reviewed By: epriestley CC: aran, Korvin, bh, jwatzman Differential Revision: https://secure.phabricator.com/D3404 |
||
|---|---|---|
| .. | ||
| behavior-accept-with-errors.js | ||
| behavior-add-reviewers-and-ccs.js | ||
| behavior-comment-jump.js | ||
| behavior-comment-preview.js | ||
| behavior-diff-radios.js | ||
| behavior-dropdown-menus.js | ||
| behavior-edit-inline-comments.js | ||
| behavior-keyboard-nav.js | ||
| behavior-populate.js | ||
| behavior-show-all-comments.js | ||
| behavior-show-field-details.js | ||
| behavior-show-more.js | ||
| behavior-toggle-files.js | ||
| behavior-user-select.js | ||
| DifferentialInlineCommentEditor.js | ||