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

Flyout Buttons with Different Font Size are Not Rendered Correctly #2793

Closed
BeksOmega opened this issue Aug 9, 2019 · 2 comments
Closed
Assignees
Labels
component: rendering component: toolbox/flyout issue: bug Describes why the code or behaviour is wrong
Milestone

Comments

@BeksOmega
Copy link
Collaborator

Describe the bug

Flyout buttons accept classes. If you give this class a font-size that is different from the standard one, the button does not render correctly.

To Reproduce

Steps to reproduce the behavior:

  1. Add the following css to the playground:
.customFlyoutButton .blocklyText {
    font-size: 30px;
}
  1. Add the following button to the playground (any toolbox category is fine):
    <button text="test" callbackKey="" web-class="customFlyoutButton"></button>
  2. Open the flyout.
  3. Observe how the button is not rendered correctly.

Expected behavior

The flyout button should render to have the correct padding around the text, regardless of the height of the text.

Screenshots

FlyoutButton_Text

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser: Chrome
  • Version: 76

Additional context

Thematically related to #2649, but the code is unrelated.

This line is the root of the problem.

@BeksOmega BeksOmega added issue: triage Issues awaiting triage by a Blockly team member issue: bug Describes why the code or behaviour is wrong labels Aug 9, 2019
@samelhusseini
Copy link
Contributor

Thanks for filing @BeksOmega. I wanted to add that this is more of a general problem with sizing based on font size that's throughout the project. You can find similar bugs with changing the font size of the text input or the .blocklyText class.

@samelhusseini samelhusseini removed the issue: triage Issues awaiting triage by a Blockly team member label Aug 9, 2019
@samelhusseini samelhusseini added this to the Icebox milestone Aug 9, 2019
@samelhusseini samelhusseini mentioned this issue Feb 19, 2020
3 tasks
@samelhusseini
Copy link
Contributor

Fixed

@samelhusseini samelhusseini self-assigned this Feb 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: rendering component: toolbox/flyout issue: bug Describes why the code or behaviour is wrong
Projects
None yet
Development

No branches or pull requests

2 participants