Skip to content

Conversation

@tvst
Copy link
Contributor

@tvst tvst commented Aug 9, 2021

(Breaking up #3642 into multiple PRs)

  • Fix funny space above the selectbox on the settings dialog, which is due to the empty widget label
  • Refactor widget labels so we handle them uniformly everywhere, making sure a label of "" does occupy space, just like today.
  • Add visual test for the empty label case above.

NOTE: Screenshot tests coming after first round of reviews. Don't want to generate and re-generate as more reviews come in 😅

NOTE #2: Note the base branch for this PR. I'm basing it on the previous UI tweaks PR so the diff is easier to read. The idea is to merge in order.

@tvst tvst force-pushed the feature/ui-4-labels branch from 148f9fd to 31ac929 Compare August 9, 2021 04:28
Copy link
Collaborator

@kmcgrady kmcgrady left a comment

Choose a reason for hiding this comment

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

I might be a little confused here. Let me know if I am missing something.

@tvst tvst force-pushed the feature/ui-3-fontfamily branch from fa5616f to cf326aa Compare August 25, 2021 05:32
@tvst tvst force-pushed the feature/ui-4-labels branch from 31ac929 to 8d52eca Compare August 25, 2021 06:16
@tvst tvst requested a review from kmcgrady August 25, 2021 16:47
@tvst tvst force-pushed the feature/ui-3-fontfamily branch from 7a7a9a2 to 732e2ac Compare August 26, 2021 06:36
@tvst tvst force-pushed the feature/ui-4-labels branch from 861381d to 1318791 Compare August 26, 2021 06:41
Base automatically changed from feature/ui-3-fontfamily to feature/ui-updates August 26, 2021 06:51
@tvst tvst merged commit 152fc4a into feature/ui-updates Aug 26, 2021
@tvst tvst deleted the feature/ui-4-labels branch August 26, 2021 06:51
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.

3 participants