Skip to content
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

Feature: Expand space in the title bars to account for longer names #1098

Merged
merged 22 commits into from
Nov 11, 2021

Conversation

ThatGalNatalie
Copy link
Contributor

@ThatGalNatalie ThatGalNatalie commented Oct 28, 2021

PR Checklist:

  • add a short description of what's changed to the top of the CHANGELOG.md
  • add/update tests (or don't, for reasons explained below)

Describe this PR

Should resolve #1082

@ThatGalNatalie ThatGalNatalie marked this pull request as ready for review October 28, 2021 22:04
Copy link

@pleek91 pleek91 left a comment

Choose a reason for hiding this comment

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

Looks good. I'm not approving as I didn't run it yet. But left a suggestion comment

src/pages/FlowRun/FlowRun.vue Outdated Show resolved Hide resolved
src/pages/TaskRun/TaskRun.vue Outdated Show resolved Hide resolved
src/pages/Flow/Flow.vue Outdated Show resolved Hide resolved
zhen0
zhen0 previously requested changes Nov 1, 2021
Copy link
Member

@zhen0 zhen0 left a comment

Choose a reason for hiding this comment

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

This looks good on larger screens but as discussed, I think you'll need to consider using breakpoints here. The name is interfering with action buttons on smaller screens. Happy to pair with you on this one if needed.

Larger screen:

Screen Shot 2021-11-01 at 2 35 41 PM

Smaller:
Screen Shot 2021-11-01 at 2 33 32 PM

Mobile:
Screen Shot 2021-11-01 at 2 33 08 PM

Current Staging - larger screen:

Screen Shot 2021-11-01 at 2 40 26 PM

@zhen0
Copy link
Member

zhen0 commented Nov 5, 2021

Screen Shot 2021-11-05 at 4 35 11 PM

Screen Shot 2021-11-05 at 4 35 20 PM

zhen0
zhen0 previously approved these changes Nov 5, 2021
Copy link
Member

@zhen0 zhen0 left a comment

Choose a reason for hiding this comment

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

Thanks @ThatGalNatalie - those updated views look good!

@zhen0
Copy link
Member

zhen0 commented Nov 11, 2021

I'm getting issues from the changelog @ThatGalNatalie. Can you remove the changelog entry and update the PR title?

@ThatGalNatalie ThatGalNatalie changed the title Expand space in title bars Feature: Expand space in the title bars to account for longer names Nov 11, 2021
Copy link
Member

@zhen0 zhen0 left a comment

Choose a reason for hiding this comment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Expand space available for Flow name
3 participants