Closed Bug 1080669 Opened 5 years ago Closed 5 years ago
Remove double-indent at resultset page boundary, optimize space
I noticed this LHF in passing. The resultset page boundary has got some (I think unwanted) double indentation due to a couple bits of markup both receiving the same padding value. In fixing we can increase page real estate a bit, and clean up the layout. I'll attach a screen grab, and put up the tweak for consideration.
See attached for the current appearance I'd like to improve.
Assignee: nobody → tojonmz
Status: NEW → ASSIGNED
Please see the above PR for review and status. Checking with Ed for feedback prior to merge.
Comment on attachment 8502688 [details] [review] treeherder-ui-PR#226 Looks good! :-)
Attachment #8502688 - Flags: feedback?(emorley) → feedback+
Commits pushed to master at https://github.com/mozilla/treeherder-ui https://github.com/mozilla/treeherder-ui/commit/045a0cfc4cdf82ecfafdbfa61ea84663a4924865 Bug 1080669 - Clean up resultset boundary and optimize page space https://github.com/mozilla/treeherder-ui/commit/2ebb18e215f28345dc1224f54e7ff8d68df1afb2 Merge pull request #226 from tojonmz/optimize-page-space Bug 1080669 - Clean up resultset boundary and optimize page space
Attachment #8502688 - Flags: review?(cdawson) → review+
Verified fixed and working correctly on dev/stage. I also made the total job count button accommodate 1,000+ job counts with this PR. They look correct also.
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Commits pushed to master at https://github.com/mozilla/treeherder https://github.com/mozilla/treeherder/commit/b85535c8bf133d6a7a5cd80a6433d4c43594878f Bug 1080669 - Clean up resultset boundary and optimize page space https://github.com/mozilla/treeherder/commit/cd2420719222fe36464675bda530c8a2ca98bc4b Merge pull request #226 from tojonmz/optimize-page-space Bug 1080669 - Clean up resultset boundary and optimize page space
You need to log in before you can comment on or make changes to this bug.