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

fixed radio button spacing with css #654

Merged
merged 7 commits into from
Apr 17, 2024
Merged

fixed radio button spacing with css #654

merged 7 commits into from
Apr 17, 2024

Conversation

000masa000
Copy link
Contributor

No description provided.

@000masa000 000masa000 requested a review from frostburn April 16, 2024 14:21
@@ -138,13 +152,13 @@ ul.btn-group {
flex-grow: 0.25 !important;
}
.control.checkbox-container {
flex-flow: unset;
flex-flow: unset;;
}
.control.checkbox-container label {
Copy link
Member

Choose a reason for hiding this comment

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

Why isn't this rule specific enough? Why the extra class?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will try your suggestion. I found I needed separate styling for the radio-group labels and the checkbox labels, so that visually they look similar.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK, it worked, thanks!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

wondering about the checkbox-group on MIDIView, would you prefer it on 2 lines (like elsewhere in the app)? I had to make a scoped style for it....

Copy link
Member

Choose a reason for hiding this comment

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

I think MIDI view deserves dedicated styles. That many checkboxes in a row is once-in-an-app exception.

@frostburn
Copy link
Member

Do all the labels. There's some you missed in e.g. ExpandScale.vue.

@frostburn
Copy link
Member

Code-wise looks good. Too many commits, but I can squash when merging on GitHub. Will review the UI after breakfast.

@frostburn
Copy link
Member

Good job! Squashing and merging.

@frostburn frostburn merged commit bcc91bf into main Apr 17, 2024
2 checks passed
@frostburn frostburn deleted the checkbox-gaps-css branch April 17, 2024 07:14
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.

2 participants