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

[BX-1057] ‘Del’ text slightly exceeds button width #1030

Merged
merged 5 commits into from
Oct 10, 2023

Conversation

derHowie
Copy link
Member

@derHowie derHowie commented Oct 3, 2023

What changed (plus any additional context for devs)

Let multicharacter button strings expand the width of the bubble keys

Screen recordings / screenshots

Screen Shot 2023-10-03 at 7 14 46 AM

What to test

Make sure button strings have sufficient horizontal padding inside the bubble keys

@linear
Copy link

linear bot commented Oct 3, 2023

BX-1057 ‘Del’ text slightly exceeds button width

brody said:

‘Del’ text slightly exceeds button width

Screenshot 2023-09-29 at 2.08.09 PM.png

@github-actions
Copy link

github-actions bot commented Oct 3, 2023

Here's the packed extension for this build:
rainbowbx-6fbc8ee190e5c1fb36d6e03f6120204265ecf809.zip

Copy link
Contributor

@estebanmino estebanmino left a comment

Choose a reason for hiding this comment

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

i feel like we should only do this when the shortcut has more than one character

for example

image

W is bigger than other letters so in the app menu these shortcut hints look misaligned

wdyt?

@derHowie
Copy link
Member Author

derHowie commented Oct 4, 2023

i feel like we should only do this when the shortcut has more than one character

for example

image W is bigger than other letters so in the app menu these shortcut hints look misaligned

wdyt?

Good call, fixed.

Screen Shot 2023-10-04 at 12 41 12 AM

@github-actions
Copy link

github-actions bot commented Oct 4, 2023

Here's the packed extension for this build:
rainbowbx-f3fde6ae0d15006ea7cc4ff6d1f85590bd5309bc.zip

@estebanmino
Copy link
Contributor

i feel like we should only do this when the shortcut has more than one character
for example
image
W is bigger than other letters so in the app menu these shortcut hints look misaligned
wdyt?

Good call, fixed.

Screen Shot 2023-10-04 at 12 41 12 AM

hm it doesn't look fixed

image

@@ -11,13 +11,14 @@ export const ShortcutHint = ({
}) => {
const height = small ? '14px' : '18px';
const width = small ? '16px' : '18px';
const widthKey = hint?.length || 0 > 1 ? 'minWidth' : 'width';
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
const widthKey = hint?.length || 0 > 1 ? 'minWidth' : 'width';
const widthKey = (hint?.length || 0) > 1 ? 'minWidth' : 'width';

@github-actions
Copy link

github-actions bot commented Oct 5, 2023

Here's the packed extension for this build:
rainbowbx-88cacc0ce329c6016008465cb0483ca21d0a991b.zip

Copy link
Member

@brunobar79 brunobar79 left a comment

Choose a reason for hiding this comment

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

🤝

@github-actions
Copy link

github-actions bot commented Oct 6, 2023

Here's the packed extension for this build:
rainbowbx-59a21f64ffbc1a5f78378a5e6591b25203d367bb.zip

@github-actions
Copy link

Here's the packed extension for this build:
rainbowbx-532314273aa64c53b5d96a083e0c4b7d72762b87.zip

@derHowie derHowie merged commit 4f08f58 into master Oct 10, 2023
15 checks passed
@derHowie derHowie deleted the @chris/expand-hints branch October 10, 2023 16:43
@github-actions
Copy link

Here's the packed extension for this build:
rainbowbx-4f08f583b6161661ad520b9475fa48015e8c3a4c.zip

@BrodyHughes BrodyHughes mentioned this pull request Oct 12, 2023
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.

4 participants