Skip to content

Conversation

rhamilto
Copy link
Member

The misalignment of the page-header bottom border and the left nav divider bugs me.

screen shot 2017-10-27 at 9 44 00 am

It's better to have them align?

screen shot 2017-10-27 at 9 43 48 am

@rhamilto rhamilto requested review from spadgett and sg00dwin October 27, 2017 13:49
@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Oct 27, 2017
@spadgett
Copy link
Member

/kind bug
/lgtm

@openshift-ci-robot openshift-ci-robot added kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. labels Oct 27, 2017
@rhamilto
Copy link
Member Author

/retest

@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue.

@openshift-merge-robot openshift-merge-robot merged commit bcff08c into openshift:master Oct 27, 2017
@rhamilto rhamilto deleted the h1-tweak branch October 27, 2017 20:14
openshift-merge-robot added a commit that referenced this pull request Nov 1, 2017
Automatic merge from submit-queue.

Adjust spacing around 'Events' heading in sidebar on monitoring page

In #2389 I adjusted the spacing around the h1 so borders aligned, but I overlooked the border underneath "Events" in the monitoring page sidebar.  This fixes that, and does so in a way that the border is consistent across browsers (it wasn't before--only Chrome had a correctly aligned border).

Before:
![screen shot 2017-11-01 at 10 46 26 am](https://user-images.githubusercontent.com/895728/32280359-f7445e58-bef1-11e7-9765-d61c6e7fe48f.PNG)

After:
![screen shot 2017-11-01 at 10 46 01 am](https://user-images.githubusercontent.com/895728/32280352-f404cbec-bef1-11e7-9c82-620523fe5814.PNG)
@rhamilto rhamilto mentioned this pull request Nov 2, 2017
openshift-merge-robot added a commit that referenced this pull request Nov 2, 2017
Automatic merge from submit-queue.

Fix bugs with headings

More cleanup of bugs I made with #2389.  I should have caught these the first two times.  But on the up side, that's one more commit for the leaderboard.  ;-)

Before:
![screen shot 2017-11-02 at 12 12 59 pm](https://user-images.githubusercontent.com/895728/32337087-6e285c56-bfc7-11e7-853f-0c0246f86aee.PNG)
![screen shot 2017-11-02 at 12 12 48 pm](https://user-images.githubusercontent.com/895728/32337088-6e34ffe2-bfc7-11e7-8019-a27fd057f58e.PNG)
![screen shot 2017-11-02 at 12 12 41 pm](https://user-images.githubusercontent.com/895728/32337089-6e3e7e5a-bfc7-11e7-8b39-0c897aa8e702.PNG)


After:
![screen shot 2017-11-02 at 12 12 06 pm](https://user-images.githubusercontent.com/895728/32337079-687fd8b0-bfc7-11e7-8054-ad158a64ddf1.PNG)
![screen shot 2017-11-02 at 12 12 14 pm](https://user-images.githubusercontent.com/895728/32337080-688ff8da-bfc7-11e7-8ac1-0d19e71fd6e5.PNG)
![screen shot 2017-11-02 at 12 12 20 pm](https://user-images.githubusercontent.com/895728/32337081-689ba860-bfc7-11e7-89d9-9540e5bcbc51.PNG)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants