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: Replace text quotes and pilcrow with custom font #5868

Closed
wants to merge 4 commits into from

Conversation

NeilFraser
Copy link
Contributor

Changing from bitmaps to vectors significantly improves the visual quality of these blocks. Also reduces the code size.

Once we have a custom font, this will allow other assets such as the dropdown arrow, the menu checkmark, the toolbox category arrows, and maybe more.

Old UI:

Screen Shot 2022-01-11 at 11 48 25

New UI:

Screen Shot 2022-01-11 at 11 48 16

Changing from bitmaps to vectors significantly improves the visual quality of these blocks.  Also reduces the code size.

Once we have a custom font, this will allow other assets such as the dropdown arrow, the menu checkmark, the toolbox category arrows, and maybe more.
FieldImage used to be used for the bitmaps.
@NeilFraser NeilFraser marked this pull request as draft January 11, 2022 23:32
@NeilFraser
Copy link
Contributor Author

Converting to a draft PR. This change has exposed deeper pre-existing issues with text-length measurement. Unsure how to fix.

@rachel-fenichel
Copy link
Collaborator

Can we do a fixed-size vector image instead of a font? It would still scale better and wouldn't need to be measured.

However, this exposes a more serious issue with Blockly which needs to be adressed first.  `getFastTextWidth` fails to honour CSS directives in the label's class.
@NeilFraser
Copy link
Contributor Author

@rachel-fenichel Let's address #5871 first, then reevaluate our position here.

@BeksOmega
Copy link
Collaborator

I'm gonna go ahead and close this since it might be a few months before we get back around to it.

@BeksOmega BeksOmega closed this Mar 28, 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.

4 participants