2023-04-28 Meeting notes

RG updates

• Merged PRs

  • npm workspaces

  • Segment event for show code example button

  • docs site updated design

  • docs: removed horizontal spacing in Button, Avatar and Badge components

  • sync master with alpha branch

  • clean up errors in tests and the follow up fix

• Resolved bunch of old tickets + done code reviews

• updated PR for removing SCSS variables in alpha branch

Tabs bug?

see this issue

Should we finish deprecating font-awesome?

created new issue here

[Adam] Is there an additional opportunity for RG to help find font-awesome usage in consuming MFEs and replace with Paragon icons to help accelerate the deprecation of font-awesome across all modern frontend repos?

[Gabe]

• Potential issue: Social media icons from font-awesome but material doesn’t.

• Is there an library for these social media icons?

PR review updates by Adam

Merged:

• [BD-46] fix: stretch the message block to the full width by monteri · Pull Request #2219 · openedx/paragon

• feat: add custom variables to utility classes by andrey-canon · Pull Request #2022 · openedx/paragon

In-progress:

• [BD-46] Remove SCSS variable usages in components and other custom styles by PKulkoRaccoonGang · Pull Request #2137 · openedx/paragon

New issues filed by Adam for RG backlog

• Create page in Open edX Confluence wiki documenting all Paragon's Segment events · Issue #2236 · openedx/paragon

• Create a "Tools" section/page on the documentation site · Issue #2237 · openedx/paragon

• Missing Paragon version in usage insights for some projects · Issue #2238 · openedx/paragon

• Improve links to code examples of component usages in "Usage Insights" · Issue #2239 · openedx/paragon

• Ensure `frontend-app-gradebook` installs `brand-openedx` by default. · Issue #2241 · openedx/paragon

• Ensure consuming projects install `@edx/brand` as the `@edx/brand-openedx` package by default · Issue #2240 · openedx/paragon

• Discovery: Paragon CLI/Migrator · Issue #2242 · openedx/paragon

• Add a "Leave feedback" hyperlink · Issue #2243 · openedx/paragon

CLI / Migration tool

Example: Polaris Migrator — Shopify Polaris

• Discovery: architecture to support CLI tool with existing design token CLI scripts?

  • Investigate Polaris CLI/Migrator’s architecture

  • Create a working prototype with at least one functional migration.
    
    

Coupled CSS builds between Paragon and @edx/brand

@Viktor Rusakov to add a comment describing the issue with example(s) on the design tokens ADR PR to collect broader input/feedback and any alternative solutions.

DataTable

• Should we abstract some of the business logic in the top-level DataTable component into a custom React hook? [Adam] yes

• react-table v6 doesn’t ship with types for TS, but v7 does. Can we upgrade?

  • [Adam] What are the breaking changes between v6 and v7? What will we have to refactor, if anything? What are the implications for consumers of DataTable?