Skip to content

Groundwork: Add OSS::TogglableSection option to remove the switch - #697

Merged
olxmpe merged 2 commits into
masterfrom
ol/vel-8177
Aug 5, 2026
Merged

Groundwork: Add OSS::TogglableSection option to remove the switch#697
olxmpe merged 2 commits into
masterfrom
ol/vel-8177

Conversation

@olxmpe

@olxmpe olxmpe commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Add an option in togglable-section component to not display the toggle switch, making sure that it does not impact current behavior

Capture d’écran 2026-08-04 à 16 59 30

Related to: #

What are the observable changes?

Good PR checklist

  • Title makes sense
  • Is against the correct branch
  • Only addresses one issue
  • Properly assigned
  • Added/updated tests
  • Added/updated documentation with Figma design link. Don't forget to replace "design" by "file" in the URL. For example https://www.figma.com/file/example
  • Migrated touched components to Glimmer Components
  • Properly labeled

@olxmpe olxmpe self-assigned this Aug 4, 2026
@linear-code

linear-code Bot commented Aug 4, 2026

Copy link
Copy Markdown

VEL-8177

@JulienVannier66 JulienVannier66 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Otherwise LGTM

Comment thread addon/components/o-s-s/togglable-section.ts Outdated
Co-authored-by: Julien "HolyDeath" Vannier <julien.vannier66@gmail.com>
@olxmpe
olxmpe merged commit e9039e2 into master Aug 5, 2026
5 checks passed
@olxmpe
olxmpe deleted the ol/vel-8177 branch August 5, 2026 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants