2023-08-04 Meeting notes

Owned by Vladyslav Zadorozhnii
Last updated: Aug 07, 2023 by Adam Stankiewicz
1 min read

 Date

Aug 4, 2023

 Participants

  • @Adam Stankiewicz

  • @Vladyslav Zadorozhnii

  • @Peter Kulko

 Goals

  •  

 Discussion topics

Time

Item

Presenter

Notes

Time

Item

Presenter

Notes

 

 

 

Merged PRs:

Reviewed PRs:

 

Update on style inhering (decoupling Paragon from *-header/footer components)

@Vladyslav Zadorozhnii

Environment: frontend-app-account
frontend-platform: slight modifications to achieve application render and have paragon core + theme, brand core + theme.
frontend-build: slight modifications to achieve brand light.min.css to appear in the <head>
Current only issue is due to the rel="preload" is not replaced to rel="stylesheet"

 

Other updates

@Vladyslav Zadorozhnii @Peter Kulko

[alpha] Active button states · Issue #2389 · openedx/paragon button states

[BD-46] feat: defined types for Paragon exports by PKulkoRaccoonGang · Pull Request #2177 · openedx/paragon typescript export

The Leave feedback link has moved down · Issue #2366 · openedx/paragon The Leave feedback link has moved down

[BD-46] refactor: fixed spacing calculations and box-shadows by PKulkoRaccoonGang · Pull Request #2423 · openedx/paragon needs discussion on strategy for deprecation

[Adam] My two cents:

  • Remove the global SCSS flags altogether. Loop it in with the breaking change for alpha anyways.

  • Create a DEPR ticket describing the removal of these global SCSS flags, what it means for consumers, and how those things should be configured in a world with design tokens.

    • Doubt there will be much push back given these global flags aren’t really used by theme authors.

[BD-46] feat: renamed custom media query by PKulkoRaccoonGang · Pull Request #2495 · openedx/paragon new opened PR for custom media queries

[Adam] Merged.

[BD-46] docs: replaced SCSS variables with CSS variables on the Colors page by PKulkoRaccoonGang · Pull Request #2376 · openedx/paragon common.css

Displaying the skeleton component · Issue #2420 · openedx/paragon Skeleton issue to current new sprint?

[Adam] Yes. Moved into current sprint.

[BD-46] feat: typescript coverage by monteri · Pull Request #2443 · openedx/paragon update on Typescript coverage.

Needs sign up and obtain token for Paragon
Enable TypeScript type coverage for PRs and `master` · Issue #2404 · openedx/paragon

[Adam] Responded. Will reach out to Axim for input.

 

New issue

@Vladyslav Zadorozhnii @Peter Kulko

Cards with `overflow: hidden` have invisible dropdowns · Issue #2498 · openedx/paragon

[Adam] D’oh, prioritized into current sprint.

 Action items

 Decisions