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

Alignment issues #2894

Closed
NeilFraser opened this issue Aug 21, 2019 · 2 comments
Closed

Alignment issues #2894

NeilFraser opened this issue Aug 21, 2019 · 2 comments
Assignees
Labels
component: rendering issue: bug Describes why the code or behaviour is wrong

Comments

@NeilFraser
Copy link
Contributor

Old renderer:
Screen Shot 2019-08-21 at 09 35 24

Current renderer:
Screen Shot 2019-08-21 at 09 35 05

There are three issues:

  1. (Blue line) Blocks on the toolbox used to be left-aligned, not counting the tab. This was a deliberate design choice. Now they are aligned including the tab.
  2. (Green line) Text on a row used to be straight. Now text height undulates between labels and fields.
  3. (Red arrow) The gap between the bottom of a statement block and its enclosing control block used to be zero if there was no "next notch" (indicating that no more blocks could be added), and the height of the "next notch" if there was one (indicating that more blocks could be added). Now there's a huge gap.

I'm also not convinced that the mutator/comment/warning icons ought to be center-aligned on the first row (new behaviour), rather than fixed to the top corner (old behaviour). These icons are block-related, not row-related.

@NeilFraser NeilFraser added issue: bug Describes why the code or behaviour is wrong component: rendering issue: triage Issues awaiting triage by a Blockly team member labels Aug 21, 2019
@rachel-fenichel
Copy link
Collaborator

  1. This is Blocks with outputs misaligned in flyout #2828
  2. Text is now center-aligned in a row (everything is center-aligned in a row) instead of absolutely positioned. Text now undulates slightly within a single block but is significantly improved within a multi-block row (and connections will follow soon):
    old: image
    new: image

We have a couple of options here:

  • Undo the vertical centering entirely
  • Define a "baseline" property and align on that instead of centering
  • Make text input fields (and fields that inherit from them) place their text slightly lower.
  1. Yup, this is a bug. Statement input sizing isn't quite right overall at this point.

Block related icons should still be aligned with the text, in my opinion. Fixing them to the corner will mean they'll be misaligned with the text in the row by a variable amount and looks weird.

@rachel-fenichel rachel-fenichel added this to the 2019_q3_release milestone Aug 28, 2019
@rachel-fenichel rachel-fenichel removed the issue: triage Issues awaiting triage by a Blockly team member label Aug 28, 2019
@RoboErikG
Copy link
Contributor

Relatively fixed. We'll take another look at text baseline alignment later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: rendering issue: bug Describes why the code or behaviour is wrong
Projects
None yet
Development

No branches or pull requests

4 participants