This extra padding was no big deal when using the current Breeze style, where scrollable
views float amorphously over the background with no clearly-defined edges, but becomes
highly noticeable in a bad way when moving towards enclosing views between single-pixel
lines, as for T11661. In this case, you want for the scrollview to touch the line.
Details
Details
- Reviewers
cblack ndavis - Group Reviewers
VDG Breeze - Maniphest Tasks
- T11661: Replace framed views with single-pixel separator lines
Diff Detail
Diff Detail
- Repository
- R31 Breeze
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 22974 Build 22992: arc lint + arc unit
Comment Actions
This LGTM visually for Dolphin, but I'm not sure how this would look when there's actually content in the areas you're adjusting pixel metrics for.
Comment Actions
Can you think of an example of where there would be? If there's content inside this margin, that seems wrong to me anyway.