Skip to content

Commit

Permalink
Merge pull request #719 from AlexVelezLl/remove-kresponsivewindowmixin
Browse files Browse the repository at this point in the history
Remove KResponsiveWindowMixin
  • Loading branch information
AlexVelezLl authored Aug 14, 2024
2 parents ce9e707 + ce9e39e commit 9f78dd4
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 199 deletions.
12 changes: 11 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,17 @@ Changelog is rather internal in nature. See release notes for the public overvie

## Upcoming version 5.x.x (`develop` branch)

- [719]
- **Description:** Removes KResponsiveWindowMixin.
- **Products impact:** removed API.
- **Addresses:** https://github.com/learningequality/kolibri-design-system/issues/679.
- **Components:** KResponsiveWindowMixin.
- **Breaking:** yes.
- **Impacts a11y:** no.
- **Guidance:** Replace any use of KResponsiveWindowMixin with the new composable useKResponsiveWindow.

[719]: https://github.com/learningequality/kolibri-design-system/pull/719

- [#718]
- **Description:** This pull request resolves failing `KDateCalendar` component tests that occurred on the last day of the month in open pull requests by setting dates manually in the tests. Additionally, the `KDateCalendar` is updated to show the month of the `lastAllowedDate` property.
- **Products impact:** none
Expand Down Expand Up @@ -35,7 +46,6 @@ Changelog is rather internal in nature. See release notes for the public overvie
- **Guidance:**
[#688] https://github.com/learningequality/kolibri-design-system/pull/688


[#707]
- **Description:** Card Validations
- **Products impact:**
Expand Down
2 changes: 1 addition & 1 deletion docs/pages/accessibility.vue
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
We should aim to allow zooming up to 200%, and down to 480x600 px in both profile and landscape. The app should always remain functional in all extreme screen sizes. Depending on the context, we may want to put additional effort into creating custom layouts optimized for particular screen sizes. For example, we should put additional effort into optimizing the mobile Learner experience, while for administrators this might not be as critical.
</p>
<p>
When designing and building layouts, we use the breakpoints defined by Material Design. These are exposed in our app by the <code>responsiveWindow</code> Vue component mixin. We should never use CSS media queries. See the <DocsInternalLink href="/layout" text="See the Layout page" /> for more information.
When designing and building layouts, we use the breakpoints defined by Material Design. These are exposed in our app by the <DocsLibraryLink component="useKResponsiveWindow" /> composable. We should never use CSS media queries. See the <DocsInternalLink href="/layout" text="See the Layout page" /> for more information.
</p>
</DocsPageSection>

Expand Down
2 changes: 1 addition & 1 deletion docs/pages/layout/index.vue
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@
<li>Level 7: <code>&gt;= 1600px</code></li>
</ul>
<p>
Responsive layouts in the design system are built using reactive JavaScript state in Vue components rather than CSS media queries. This is done using <DocsLibraryLink component="KResponsiveWindow" /> when reactive window's size information is needed or <DocsLibraryLink component="KResponsiveElement" /> when reactive component's size information is needed.
Responsive layouts in the design system are built using reactive JavaScript state in Vue components rather than CSS media queries. This is done using <DocsLibraryLink component="useKResponsiveWindow" /> when reactive window's size information is needed or <DocsLibraryLink component="KResponsiveElement" /> when reactive component's size information is needed.
</p>
</DocsPageSection>

Expand Down
196 changes: 0 additions & 196 deletions lib/KResponsiveWindowMixin.js

This file was deleted.

0 comments on commit 9f78dd4

Please sign in to comment.