Skip to content

Conversation

HeleenSG
Copy link
Contributor

@HeleenSG HeleenSG commented Jul 9, 2025

Implements a version where we use sass variables instead of css custom properties.

Todo (before merging to main

#996

  • Snelstart: Testen (Rieven)
  • Tutorial: Voorbeelden updaten (Heleen)
  • Nota bene: "@use "@minvws/manon/text/nota-bene-variables";" weg (chekcen Heleen)
  • Dropdown button: Testen en kijken waar we staan. Styling lijkt te missen (Rieven)
  • Collapsible (Rieven)
  • Fieldset horizontal: Fieldset horizontal behaviour isn't working (Heleen)

Iconen

  • Icoonknoppen zonder achtergrond
  • Icoonknoppen zonder achtergrond: Hebben wel een achtergrond. Testen en kijken waar we staan.
  • Icons (mixin) opruimen
  • Help: Icoon en uitlijning

Todo:

  • Aan de slag: Korte uitleg mbt keuze toevoegen ("ik weet wat ik moet doen" etc)
  • Snelstart: Introductie header toevoegen
  • Installation: Testen
  • Installation: Fonts in themadocumentatie
  • Subtielere weergave en Benadrukte weergave: Titel op pagina komt niet overeen met tekst in de link op componentenpagina
  • Headings: Pagina oogt druk
  • Utility classes: Introductie toevoegen aan utility classes
  • Focus only: notification uitlijning (code)
  • Call to action button: Styling toevoegen
  • Nota bene: Lijkt nu alleen voor invoervelden te zijn maar kan ook los gebruikt worden. Update voorbeelden en documentatie
  • Horizontal: "voorbeeld: Object met horizontaal uitgelijnde elementen" gaat mis
  • Background color offset: Documentatie toevoegen
  • Dropdown button: pagina gaat nog niet goed.
  • Code & Code block: Documentatie mist en styling mist
  • Filter: Styling mist

Tabs

  • Overflow styling / behaviour
  • Tabpanel styling (if any)
  • Nice to have:
    • Configurable activation mode (e.g. <manon-tabs activation="manual">)
    • window.location awareness (update when switching, initially selected tab on load)
    • Asynchronous content loading (support in JS? or just documentation?)

Button
Knoppen (basisweergave)
Call to action button

  • Introductietekst mist

Breadcrumbs

  • Titels niet consistent
  • Introductie mist. Is nu enkel "Breadcrumb navigatie"
    "Breadcrumb-navigatie verteld de gebruiker waar hij zich bevind binnen de applicatie. Hiermee kan de gebruiker de weg terugvinden en sneller navigeren."
  • Plaatsing in de html beschrijven

Designvragen

  • Sidemenu voor componenten en utility classes ipv toc
  • Sidebar is niet sticky
  • Bij componenten link naar figma toevoegen?

Destructief

  • Bijbehorende bestanden weg
  • Instelbare variabelen weg
  • Gerelateerde pagina's weg
  • Disabled voorbeeld weg

Description list

  • Styling mist (Checken of het al gedaan is, Heleen)

Card

  • Titel inconsistent
  • Styling mist

Logo

  • Styling mist
  • Visueel voorbeeld mist
  • Bijbehorende bestanden weg
  • Quickstart toevoegen

Terug naar hoofdmenu knop

  • Styling
  • Bijbehorende bestanden weg
  • Quickstart toevoegen

Overkoepelend

  • Nice to have: copy icoon bij voorbeelden
  • Spacing fixen
  • Sidemenu nalopen op alle pagina's en gelijk trekken qua content (in ieder geval voor de componenten?)
  • Disabled states weghalen (is een ticket voor. Heleen)
  • Remove mixins that will become obsolete

Icoonknoppen

  • Button base > button
  • Testen en kijken waar we staan.

Breaking changes

  • Usage of sass variables instead of css variables. If you reference Manon CSS variables they will no longer exist. Use the new sass variables instead.

Layout

  • Header, content wrapper now uses a class content-wrapper on the div or section. This is added to prevent css to be implemented too broadly.

Components

  • Simplified styling options for table head. From now on all styling done on table-head and table-head-cell will be added as styling to table-head. For example: $table-head-cell-padding is now $table-head-padding.
  • label and input are now wrapped in a
    to be able to define spacing.
  • Form help now uses a class to determine the parent to which the button is aligned. Define the parent by adding the class "help-container". Alignment is now set on the selected parent instead of the entire form.

Renamed

  • "horizontal-center" to "horizontal-centered" to be in line with "centered".
  • "form-input-color" to "color-picker" and merged "form-input-color-filled" into "color-picker.
  • button-ghost to button-secondary
  • breadcrumb-content-block* to breadcrumb-bar-content-wrapper

Removed "base" from component names such as "button-base" simplified now to "button".

  • form-base to form
  • form-required-base to form-required
  • button-base to button
  • application-base to base
  • list-base to list
  • Message-counter-base to message-counter
  • "centered-view". Use "centered" instead.
  • "form-radio". Use "radio" instead.
  • "horizontal-view" to "horizontal"
  • "fity-fifty". Use columns (column-2) instead

Removed

  • "footer-one-third-two-thirds". Use the utility layout classes instead.
  • layout div and layout div-content-wrapper. Use main, section or article instead.
  • layout-authentication use container-s, container-m or container-l instead.
  • layout-form use container-s, container-m or container-l instead.
  • layout-set. Shared layout values can be set per theme.
  • Color schemes have been moved out of Manon and can be added separate per theme.
    Colors:
    • bg-color-offset
    • critical-informational
    • notification-colors
    • finn (large, medium and small)
    • spot (large, medium and small)

Merged

Merged content wrapper styling options in with their parent component. And added the class content-wrapper to the target element. This to prevent too many separate files to include.

  • article-content-wrapper merged with article
  • main-content-wrapper merged wit main
  • section-content-wrapper merged with section
  • breadcrumb-bar-content-wrapper merged with breadcrumb-bar

Solves

#421 (for footer)
#186

@HeleenSG HeleenSG self-assigned this Jul 9, 2025
@Rieven Rieven force-pushed the feat/manon-sass branch from d02e436 to ddbf006 Compare July 14, 2025 11:19
ppvg and others added 29 commits September 11, 2025 18:39
* build: ignore empty fragment URLs

* build: don't ignore missing IDs

* chore: update placeholder fragment URLs
* chore(deps): add Testing Library

* feat: interactive tabs component

* chore: add description to initTab's jsdoc

Co-authored-by: JP Bruins Slot <[email protected]>

* refactor: initTab

Co-authored-by: JP Bruins Slot <[email protected]>

* chore: add description to handleClick's jsdoc

Co-authored-by: JP Bruins Slot <[email protected]>

* chore: add description to handleKeyDown's jsdoc

Co-authored-by: JP Bruins Slot <[email protected]>

* chore: add description to setActiveTab's jsdoc

Co-authored-by: JP Bruins Slot <[email protected]>

* chore: make jest-dom matcher types available to typescript

* fix: support tabs with inner element

* fix: perform DOM setup in connectedCallback instead of constructor

* chore: run formatter

* docs: update tabs documentation

* chore: move JS tests to manon/tests/ alongside sass tests

* chore: add dev dependency @types/node

* fix: docs js import syntax error

---------

Co-authored-by: JP Bruins Slot <[email protected]>
# Conflicts:
#	examples/sass-cli/package.json
#	pnpm-lock.yaml
…#1065)

* docs: remove {base} from the markdown files in `docs`

* docs: add `remarkLinks` plugin to rewrite `<a>` tags in markdown files

* docs: replace `{base}` placeholder with absolute paths in markdown files

* docs: update comments in `remarkLinks`

* docs: fix indentation in navigation component example code
* Fixes sticky table header and dolor set to dolor sit

* fixing overcorrection

* style: remove trailing whitespace in `table-sticky-header.scss`

* style: fix indentation in navigation component example code

---------

Co-authored-by: Randhir Gulraj <[email protected]>
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
2 Security Hotspots
0.0% Coverage on New Code (required ≥ 80%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

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.

Merge manon-sass into the main branch
6 participants