Skip to content

fix(ui) Fix short-id overlap on issue details #12343

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Mar 12, 2019
Merged

Conversation

markstory
Copy link
Member

A long single-word project slug would have overlap issues in smaller viewports due to flexbox shrinkage which is now disabled.

Example of problem

screen shot 2019-03-08 at 1 06 38 pm

A long single-word project slug would have overlap issues in smaller
viewports due to flexbox shrinkage which is now disabled.
@markstory markstory requested review from Chrissy and a team March 8, 2019 18:12
Updating the component globally causes problems on project overview
<StyledProjectBadge
project={project}
avatarSize={20}
hideName={true}
Copy link
Member

Choose a reason for hiding this comment

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

You can leave off the ={true} part of this.

@markstory markstory merged commit 743423f into master Mar 12, 2019
@markstory markstory deleted the issue-slug-overlap branch March 12, 2019 13:54
jan-auer added a commit that referenced this pull request Mar 12, 2019
* master:
  fix(ui) Fix short-id overlap on issue details (#12343)
  fix(ui) Going to a global saved search from a project based one (#12358)
  feat(hooks): Add named hooks to events / discover Features (#12365)
  test(ui): Add test for LazyLoad props change (#12366)
  test(js): Update snapshots from Button changes (#12371)
  fix(tags): Remove timestamp tag support from issue stream (#12353)
  meta(vscode): Do not autoformat Django templates (#12350)
@github-actions github-actions bot locked and limited conversation to collaborators Dec 20, 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.

2 participants