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

#3 Created the Calendar Layout #33

Merged
merged 5 commits into from
Dec 5, 2022

Conversation

madelahn
Copy link
Contributor

Developed the basic calendar layout in CalendarScreen.kt.
The calendar is able to vertically scroll from the current month to 12 months before, with the labels for the day of the week and each month displayed as the header for each month.

Some notes and things to implement:

  • @OptIn(ExperimentalPagerApi::class) is added to every function, since the minimum SDK requirements for using the Calendar library requires a version above 21 (where the current min. SDK version is 21). I can set it to something higher but I just need confirm it won't conflict with anything else.
  • Look into how create separate pages for each date.
  • Fix the UI to look more similar to the original app.

@madelahn madelahn linked an issue Nov 30, 2022 that may be closed by this pull request
4 tasks
@madelahn madelahn self-assigned this Nov 30, 2022
@madelahn madelahn linked an issue Nov 30, 2022 that may be closed by this pull request
2 tasks
@michaelng0107 michaelng0107 changed the base branch from develop to calendar-nav November 30, 2022 13:55
@michaelng0107
Copy link
Contributor

Changed base branch so that it only shows @madelahn's changes

@madelahn madelahn marked this pull request as ready for review December 4, 2022 21:42
@madelahn madelahn merged commit 18909c4 into calendar-nav Dec 5, 2022
madelahn added a commit that referenced this pull request Dec 5, 2022
madelahn added a commit that referenced this pull request Dec 5, 2022
madelahn added a commit that referenced this pull request Dec 5, 2022
leowrites pushed a commit that referenced this pull request Dec 5, 2022
* Added Basic Calendar Cycle Top Bar

* added dependencies

* Styled Calendar Cycle Top Bar

* Changed TopBar to use tabs, added navigation

* setup basic navigation bar and graph

* added calender navgiation to FAB

* modularized BottomNav

* modularized FAB

* moved Screen enum to NavigationGraph

* Added Swiping to tabs, comments

* cleanup

* removed unused icon

* Updated Indicator Color

* removed unused colors

* added dependencies and tests

* Added activity to manifest - temp fix so tests run

Was seeing the following issues when I run tests in NavigationTest:
- android.content.ActivityNotFoundException: Unable to find explicit activity class {com.example.theperiodpurse.test/androidx.test.core.app.InstrumentationActivityInvoker$EmptyActivity}; have you declared this activity in your AndroidManifest.xml, or does your intent not match its declared <intent-filter>?
- android.content.ActivityNotFoundException: Unable to find explicit activity class {com.example.theperiodpurse.test/androidx.test.core.app.InstrumentationActivityInvoker$BootstrapActivity}; have you declared this activity in your AndroidManifest.xml, or does your intent not match its declared <intent-filter>?

Referred to android/android-test#196 for the fix.
Looks like this is fixed in androidx.test:core:1.5.0-alpha02.

* removed changes to manifest

* added androidTestResultsUserPreferences to gitignore

* added useful contentDescription to FAB

* added instrumental tests to CI

* added emulator use to workflow

* fixed typo in ci

* test with macOS

* specify system image

* Added content description for testing

* Added content description for testing

* Added testing for tabs

* added check to hide navigation bar during onboarding

* better implementation for hiding nav bar

* bandaid fix for tests

* fix tests

* create different screen to start in Calendar page

* added skipOnboarding option to AppScreen

* popback stack after onboarding

* #3 Created the Calendar Layout (#33)

* Added basic calendar setup to the screen

* Changed the minSDK in Project Configs

* Updated Visuals to look like original app

* Added Click Functionality

* Revert "#3 Created the Calendar Layout (#33)" (#38)

This reverts commit 18909c4.

Co-authored-by: Pierre-William Lessard <lessardpw@gmail.com>
Co-authored-by: Madeline <93456777+madelahn@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement the calendar on the calendar page Create the calendar page layout
2 participants