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

Keep text in InputConfirmCancel static #2779

Merged
merged 2 commits into from
Jun 20, 2022

Conversation

raimund-schluessler
Copy link
Contributor

@raimund-schluessler raimund-schluessler commented Jun 17, 2022

@jancborchardt Since you mentioned this in the chat, this fixes the jumping text when switching to edit mode in AppNavigationItem.

Before:
Item_before

After:
Item_after

@nextcloud/designers Please note the tradeoff, that the left border of the input field is now closer to the icon than before. I am not sure this is wanted, but I think it's the only way to keep the text static without making it look weird when the input is not shown.

Signed-off-by: Raimund Schlüßler <[email protected]>
Signed-off-by: Raimund Schlüßler <[email protected]>
@raimund-schluessler raimund-schluessler added this to the 6.0.0 milestone Jun 17, 2022
@raimund-schluessler raimund-schluessler added enhancement New feature or request 3. to review Waiting for reviews design Design, UX, interface and interaction design discussion Need advices, opinions or ideas on this topic labels Jun 17, 2022
@raimund-schluessler raimund-schluessler marked this pull request as ready for review June 17, 2022 08:22
@marcoambrosini marcoambrosini merged commit 7838efc into master Jun 20, 2022
@marcoambrosini marcoambrosini deleted the fix/noid/input-text-static branch June 20, 2022 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews design Design, UX, interface and interaction design discussion Need advices, opinions or ideas on this topic enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants