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

Label for switch is not vertically aligned #1072

Closed
3 tasks done
Lausselloic opened this issue Jan 31, 2022 · 2 comments · Fixed by #1081
Closed
3 tasks done

Label for switch is not vertically aligned #1072

Lausselloic opened this issue Jan 31, 2022 · 2 comments · Fixed by #1081
Labels

Comments

@Lausselloic
Copy link
Member

Prerequisites

Describe the issue

https://boosted.orange.com/docs/5.1/forms/checks-radios/#switches
Label is not well aligned
image

Reduced test cases

https://boosted.orange.com/docs/5.1/forms/checks-radios/#switches

What operating system(s) are you seeing the problem on?

Windows

What browser(s) are you seeing the problem on?

Firefox

What version of Boosted are you using?

91.2.0 ESR

@Lausselloic
Copy link
Member Author

image
Jumping padding-top for
.form-switch .form-check-label to 0.413rem could do the job?

Need a Design validation

@julien-deramond
Copy link
Member

julien-deramond commented Jan 31, 2022

Thanks for the issue!

Jumping padding-top for .form-switch .form-check-label to 0.413rem could do the job?

Yes, it seems to do the job.

Need a Design validation

Asked to the designers: the label for the switches can be vertically centered.

klediouron pushed a commit that referenced this issue Feb 11, 2022
author LE DIOURON Kevin <kevin.lediouron.ext@orange.com> 1626946275 +0200
committer LE DIOURON Kevin <kevin.lediouron.ext@orange.com> 1644573012 +0100

parent c27e1e1
author LE DIOURON Kevin <kevin.lediouron.ext@orange.com> 1626946275 +0200
committer LE DIOURON Kevin <kevin.lediouron.ext@orange.com> 1644572436 +0100

feat(navbars): add Global Headers components

chore(review): fix bad URLs, JS formatting and SCSS errors

chore(review): remove .global class reference in migration page

fix(misc): WIP reviews from Louis-Maxime

fix(misc): WIP reviews from Louis-Maxime

fix(logo): correct scss for imgs & svgs

fix(headers): align on the bottom of the text

Fix(Examples) : in large viewport, orange bar below active item located to high + extra border in Exampes/NavBar (OBS)

Signed-off-by: Isabelle Chanclou <isabelle.chanclou@orange.com>

Fix(Examples) : in small viewport, orange bar below active item located to high

Signed-off-by: Isabelle Chanclou <isabelle.chanclou@orange.com>

Fix(global header) : add collapse for viewport 768-1023px

Signed-off-by: Isabelle Chanclou <isabelle.chanclou@orange.com>

fix(navbar): spacing between last-child

fix(minimized): [wip] spacing between icons, remove important later

fix(style): [wip] clean navbar css

review: add navbar static and fixed examples

fix(scss) : [WIP] clean css

fix(scss): [WIP] clean orange-navbar

fix(scss): change comment to disable important check

Fix(orange-navbar): height of global header and all its variants have a correct height in small and larger breakpoints

Signed-off-by: Isabelle Chanclou <isabelle.chanclou@orange.com>

Fix(orange-navbar): height of global header and all its variants have a correct height in small and larger breakpoints

Signed-off-by: Isabelle Chanclou <isabelle.chanclou@orange.com>

fix(html): Revamp of the html for examples. Some checks to do (height of elements, ...)

fix(html): shortcode template for orange-navbar examples

fix(menus): Ellipsis for when a text is wider than 8rem

Fix for responsive part to fit to others atm

Amend the large viewport container misalignment

Sticky header example (#1089)

* Add(Examples): add a sticky global header + supra bar example

Signed-off-by: Isabelle Chanclou <isabelle.chanclou@orange.com>

* fix(misc): Shenanigans to fix css-side

* Backport some changes

* Minor fixes

* fix(fixed): Have the supra + global properly fixed and sticky at the top. Todo : top 40px (into rem)

Co-authored-by: Isabelle Chanclou <isabelle.chanclou@orange.com>
Co-authored-by: LE DIOURON Kevin <kevin.lediouron.ext@orange.com>

fix(sticky): Allocate some height to display both supra + global navbars with stickiness

fix(nav-link): Show anew the active selector

fix(nav-link): revert to the "original" padding as in the main branch

Can't even remember why the padding in line 203 has been changed...

Icons spacing + icons alignment

fix(supra): Change the padding-bottom variable to fix the missing pixel

Revert navbar-fixed and navbar-static examples

fix(sticky): Fix the example to have both navbars into the header markup

Add fake navbar-sticky images to fix 'npm run docs'

Minimized fixed + burger

Fix example

fix(sticky): remove grey container and banner role

fix(overflow): Revert the overflowing of content since it doesn't suit us atm

Small changes in md- breakpoints

fix(breakpoint): menu for doc navbar compatible with md breakpoint

md breakpoint

.

Minor fix on supra + .active

fix(docs): Use shortcodes for examples, add a condition to hide basket icon. TODO (or not ?): fix wrapping in docs

Minor fixes

chore(merge main) patched commit → c1222d6 (#1063)

feat(spinners): reintegrate growing spinners from v4 (#1041)

Signed-off-by: louismaximepiton <louismaxime.piton@orange.com>
Co-authored-by: Julien Déramond <julien.deramond@orange.com>

chore(dev-deps): drop `delay-cli` and add `sirv-cli` (#1005)

chore(merge main) patched commit → 76802e2 (#1071)

fix(#1072): vertically align labels for switches (#1081)

fix(forms): black outline for focused switches (#1082)

chore(merge main) patched commit → dc09509 (#1080)

fix(accordion): remove active text color on focus (#1087)

chore(merge main) patched commit → a805330 (#1090)

chore(merge main) patched commit → 8f11c52 (#1092)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants