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

fix(tagging): align error text with save button on edit link drawer #2029

Merged
merged 1 commit into from
Oct 13, 2022

Conversation

halfwhole
Copy link
Collaborator

@halfwhole halfwhole commented Oct 13, 2022

Problem

See issue description in #2026

Closes #2026

Solution

On the edit link form, ensure that the tags field is aligned with the save button, by fixing the helper text's position so that it does not take extra height.

However, we need a way to toggle between making the helper text add extra height on the create link form, and adding no height on the edit link form. This is done through the use of the boolean variable fixHelperTextPosition.

Screenshots

Before:

Create link form

Screen.Recording.2022-10-13.at.4.19.47.PM.mov

Edit link form

Screen.Recording.2022-10-13.at.4.20.57.PM.mov

After:

Create link form: unchanged from before

Edit link form

Screen.Recording.2022-10-13.at.4.24.02.PM.mov

@halfwhole halfwhole requested a review from thanhdatle October 13, 2022 03:11
Copy link
Contributor

@thanhdatle thanhdatle left a comment

Choose a reason for hiding this comment

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

LGTM

@halfwhole halfwhole merged commit 0b9d58e into develop Oct 13, 2022
@halfwhole halfwhole deleted the fix/tagging/align-error-text branch October 13, 2022 09:38
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.

Link tagging: tags input and save button are not aligned when user enters incorrect tag input
2 participants