2025-05-21 Meeting notes

2025-05-21 Meeting notes

All public Working Group meetings follow the Recording Policy for Open edX Meetings

 Date

May 21, 2025

 Participants

  • @Adam Stankiewicz

 Goals

  •  

 Discussion topics

Time

Item

Presenter

Notes

Time

Item

Presenter

Notes

 

 

@Adam Stankiewicz @Kira Miller

Sanity check withDeprecatedProps usage within ProductTour:

feat: changing the header of ProductTour by kiram15 · Pull Request #3572 · openedx/paragon

tl;dr;

  • The existing dismissButtonText is deprecated, and migrated to dismissAltText.

  • We’ve never used withDeprecatedProps to deprecate sub-properties of a top-level component prop, as we are needing to with ProductTour.

  • Any concerns with the approach in the above PR?

Notes:

  • Only (blocking) feedback before approval/merge, we should output a console.warn when we ignore a non-string prop type in the transform passed to withDeprecatedProps to ProductTour.

  • @Kira Miller to also add an additional sentence in the “usage guidelines” around consistent title usage throughout

  • @Brian Smith to make issue for A11y in regards to using headings for orienting to steps/discovery around other possible ways. Also look into where focus returns upon completion of the tour.

  • Concerns around tour checkpoints that may bring users to a new page route (e.g., with client-side routing), without being explicit/clear about the navigation.

    • Impacts both next & back.

      • E.g., button vs. a

 

Overriding default button variants in Alert via @edx/brand

@Adam Stankiewicz

feat: introduce tokens approach to overriding alert btn variants by adamstankiewicz · Pull Request #3598 · openedx/paragon

Notes:

  • Should at least document in @edx/brand-openedx README.



Some open design tokens PRs

@Brian Smith

 Action items

 Decisions