Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[JetNews] Update window size breakpoints #658

Merged
merged 1 commit into from
Sep 23, 2021
Merged

Conversation

alexvanyo
Copy link
Contributor

Updates the window breakpoints, mainly be widening the range that is considered MEDIUM.

@alexvanyo alexvanyo merged commit 15baabf into ls Sep 23, 2021
@alexvanyo alexvanyo deleted the av/840dp-window-size branch September 23, 2021 16:27
manuelvicnt pushed a commit that referenced this pull request Oct 13, 2021
* [Jetnews] Snackbars large screens support (#631)

* [Jetnews] Add WindowSize breakpoints (#632)

* [Jetnews] Add NavRail for large screens (#636)

* Add initial large screen exploration

* PR feedback and refine integration with navrail

* Fix formatting and tests

* Use rememberUpdatedState for notifyInput

* Add remember for windowSize

* [Jetnews] Interests screen LS support (#639)

* [Jetnews] Large screen support for the Interests tab (#641)

* [JetNews] Fix state saving with drawer (#653)

* [JetNews] Fix state saving with drawer

* Extract into rememberSizeAwareDrawerState

* [JetNews] Convert navrail/drawer decision to use currentWindowMetrics (#654)

* [JetNews] Convert navrail/drawer decision to use currentWindowMetrics

* Move window metrics helpers to WindowSize.kt

* [Jetnews] Better TopAppBar in large screens (#650)

* [Jetnews] Move NavRail to top level component (#655)

* [JetNews] Update window size breakpoints (#658)

* [JetNews] Adjust medium behavior (#657)

* [JetNews] Adjust medium behavior

* Move column decision out of BoxWithConstraints

* Follow my own rule for isDrawerActive

* Update for PR feedback

* Spotless apply

* Add TODO for BoxWithConstraints usage

* [JetNews] Replace currentWindowMetrics flow with synchronous calculation (#660)

* [Jetnews] Consolidate compact and medium layouts (#659)

* [Jetnews] Add space between NavRail icons (#663)

* [Jetnews] Custom interests layout (#665)

* [JetNews] Move window size calculation to activity level (#667)

* [JetNews] Move window size calculation to activity level

* Re-order WindowSize functions

* Fix formatting

* Merge main into ls (#672)

* Implemented swipe to delete for Jetsnack cart

* Replace logic for SwipeToDismiss composable

Added Animated visibility as well

* Requested adjustments

* Removed Opt in

* Spotless apply

* PR fixes

* Swipe to delete polish

* Spotless

* Removed unnecessary Row

* Code polish

* feat : improve UX by setting Button semantic role to play image button

fix : fix not beeing able to move from CategoryPodcast to EpisodeList with Talkback

feat: merge podcast category card with podcast title

* Break long lines

* Content to trailing lambda, added comments

* Fix invalid neutral color #591

* [Jetnews] Showcase PreviewParameterProvider usage.

* [Jetnews] Move PostPreviewParameterProvider.kt and add documentation

* Fix kdoc format

* [Jetsnack] Snackbar support (#645)

* Finish checksum.sh sentence (#622)

* [All] Update to Compose 1.0.2

* Spotless apply

* [All] Revert to previous version of ktlint

* [Jetsnack] Add state holder to JetsnackApp (#646)

* fix : fixing formating issue

* Update Compose Material Catalog AOSP link (#656)

* [Crane] Use createEmptyComposeRule for test with injection

* [All] Update to Compose 1.0.3 (#671)

* [Jetnews] Update to Compose 1.1.0-alpha05

* [Jetnews] Spotless

Co-authored-by: Angeles <[email protected]>
Co-authored-by: DEMEY Fanny <[email protected]>
Co-authored-by: Chris Sinco <[email protected]>
Co-authored-by: Jolanda Verhoef <[email protected]>
Co-authored-by: Jose Alcérreca <[email protected]>
Co-authored-by: Alex Vanyo <[email protected]>
Co-authored-by: Nick Rout <[email protected]>

* [Jetcaster] Add tabletop posture support to the Player screen (#676)

Co-authored-by: Alex Vanyo <[email protected]>
Co-authored-by: Angeles <[email protected]>
Co-authored-by: DEMEY Fanny <[email protected]>
Co-authored-by: Chris Sinco <[email protected]>
Co-authored-by: Jolanda Verhoef <[email protected]>
Co-authored-by: Jose Alcérreca <[email protected]>
Co-authored-by: Nick Rout <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants