Skip to content

fix(ui) Fix clipped descenders on project badge.#10687

Merged
markstory merged 3 commits into
masterfrom
project-descender
Nov 21, 2018
Merged

fix(ui) Fix clipped descenders on project badge.#10687
markstory merged 3 commits into
masterfrom
project-descender

Conversation

@markstory

Copy link
Copy Markdown
Member

Add a smidgen more line-height to display clipped descenders.

Before

screen shot 2018-11-20 at 1 49 23 pm

After

screen shot 2018-11-20 at 1 48 58 pm

Add a smidgen more line-height to display clipped descenders.
@markstory markstory requested a review from a team November 20, 2018 19:05

@billyvg billyvg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@evanpurkhiser

Copy link
Copy Markdown
Member

The alignment of settings headers gets a little worse after this :(

@markstory

Copy link
Copy Markdown
Member Author

@evanpurkhiser Are you referring to the badge/title placement changes? I thought having the badges shifted down 2px was nicer as the weight of the badge is more towards the center.

@markstory markstory merged commit b286fde into master Nov 21, 2018
@markstory markstory deleted the project-descender branch November 21, 2018 16:16
jan-auer added a commit that referenced this pull request Nov 22, 2018
* master:
  feat: Initial pass on legacy message removal (#10713)
  fix: Crash when generating relay PII config (#10716)
  fix(gcs): Catch down transport errors for reconnects (#10726)
  feat: Add retry and timeout workaround for GCS failures (#10717)
  fix(snuba): Fix search pagination, remove old/unused in-memory scorin… (#10718)
  fix(features): Clean up auth provider component renderers (#10721)
  tests(ui): Wrap data forwarding with data-test-id (#10704)
  fix(snuba): Return empty results (rather than AssertionError) for out-of-range searches (#10719)
  fix(discover): move sidebar to the left, improve negative space, fix header vibrations  (#10709)
  feat(discover): Result table responds to window size changes (#10692)
  feat(ui): beta badge (#10672)
  feat(forms): Allow entire forms to be disabled (#10707)
  feat(discover): Bump up max visible rows on table (#10702)
  fixed case in default repos where there are no repos. (#10706)
  fix(integrations): Azure DevOps check all users in outbound assignee sync. (#10667)
  fix(ui) Fix clipped descenders on project badge. (#10687)
  feat(jiraserver) Add OAuth connection process (#10685)
  fix(integrations) Include more context in logs when commit fetch fails (#10690)
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants