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

Fixes #2583 Title container overflow fixed #2624

Closed
wants to merge 1 commit into from

Conversation

irenejoeunpark
Copy link
Contributor

@irenejoeunpark irenejoeunpark commented Dec 16, 2021

Issue This PR Addresses

Fixes #2583

Type of Change

  • Bugfix: Change which fixes an issue
  • New Feature: Change which adds functionality
  • Documentation Update: Change which improves documentation
  • UI: Change which improves UI

Description

Setted overflowY as hidden to avoid title container overflow.

overflow.mp4

Checklist

  • Quality: This PR builds and passes our npm test and works locally
  • Tests: This PR includes thorough tests or an explanation of why it does not
  • Screenshots: This PR includes screenshots or GIFs of the changes made or an explanation of why it does not (if applicable)
  • Documentation: This PR includes updated/added documentation to user exposed functionality or configuration variables are added/changed or an explanation of why it does not(if applicable)

@gitpod-io
Copy link

gitpod-io bot commented Dec 16, 2021

Copy link
Contributor

@DukeManh DukeManh left a comment

Choose a reason for hiding this comment

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

@irenejoeunpark is this linked with #2583?

@irenejoeunpark
Copy link
Contributor Author

@irenejoeunpark is this linked with #2583?

I added the linked issue! Thanks

@AmasiaNalbandian
Copy link
Contributor

@irenejoeunpark Can we rebase this so it's ready for merge? :)

@irenejoeunpark
Copy link
Contributor Author

@irenejoeunpark Can we rebase this so it's ready for merge? :)

I have fixed all of the conflicts caused while rebasing, however I am getting error with the pnpm commands which blocks me to commit. I know it is not a good practice, but can I fix the conflicts by editing in github for this one?

@DukeManh
Copy link
Contributor

@irenejoeunpark, fixing conflicts here would merge master in your branch, which is not what we want. If u have problems with your local environment, I'd recommend taking a look at our Gitpod docs and continue your work on there.

@AmasiaNalbandian
Copy link
Contributor

@irenejoeunpark, fixing conflicts here would merge master in your branch, which is not what we want. If u have problems with your local environment, I'd recommend taking a look at our Gitpod docs and continue your work on there.

Thanks Duke! @irenejoeunpark It seems easier but it really does make it messier. I will be around if you need help with this :)

@Kevan-Y
Copy link
Contributor

Kevan-Y commented Jan 13, 2022

Hi @irenejoeunpark, could you update this branch to the latest. Thanks

@Kevan-Y Kevan-Y self-requested a review January 13, 2022 03:10
@humphd humphd added this to the 2.5 Release milestone Jan 13, 2022
@TDDR TDDR mentioned this pull request Jan 18, 2022
4 tasks
@TDDR TDDR closed this Jan 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Title container floating with some space, and overflow with the scroll on mobile view.
8 participants