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

General principles page #10

Merged
merged 9 commits into from
Feb 13, 2024
Merged

General principles page #10

merged 9 commits into from
Feb 13, 2024

Conversation

PiXlRAM
Copy link
Member

@PiXlRAM PiXlRAM commented Feb 2, 2024

Tracking Info

Resolves #5

Changes

Created General Principles and Emergency Care pages, created bullet point component that is used in a lot of other pages with the same format at General Principles (such as Emergency Care), used nested list view for multiple subpoints

Testing

Checked the page on iOS device and Android virtual device, both of which were different screen sizes too, worked fine in all cases.

Confirmation of Change

image

@Anthonyp0329 Anthonyp0329 requested review from a team, sydneywb and LillianHo5 and removed request for a team February 2, 2024 01:34
Copy link
Collaborator

@Anthonyp0329 Anthonyp0329 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks awesome, great work guys, especially with making the bullet point component. Just a few changes before this is ready to merge - make sure to put the styles into a separate file for each component, so BulletPointStyles, generalPrinciplesStyles, etc. Also, remove any changes to App.tsx for now. Then, should be good to go! Thanks!

@Anthonyp0329 Anthonyp0329 requested review from a team, shreyagupta112, meganleongg and eshaan-s18 and removed request for sydneywb, a team, shreyagupta112 and meganleongg February 5, 2024 06:22
Copy link
Contributor

@LillianHo5 LillianHo5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me! Aside from what Anthony mentioned, one thing I noticed was this error message showing up when running the IOS/Android emulators:

ERROR fontFamily "Roboto" is not a system font and has not been loaded through Font.loadAsync.

  • If you intended to use a system font, make sure you typed the name correctly and that it is supported by your device operating system.
  • If this is a custom font, be sure to load it with Font.loadAsync.

I also had this error with my task, and I thought that this thread and docs were helpful: https://stackoverflow.com/questions/57066075/console-error-fontfamily-roboto-medium-is-not-a-system-font-and-has-not-been, https://docs.expo.dev/develop/user-interface/fonts/#using-fontloadasync-instead-of-the-usefonts-hook.

Copy link
Contributor

@eshaan-s18 eshaan-s18 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks awesome guys! Great job, especially with those bullet point components. Aside from what Anthony and Lillian mentioned, it appears to run well for me. Approving this PR, great job guys!

Copy link
Collaborator

@Anthonyp0329 Anthonyp0329 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing the linting issues, is good to merge!

@PiXlRAM PiXlRAM merged commit 2473622 into main Feb 13, 2024
2 checks passed
@PiXlRAM PiXlRAM deleted the general-principles-page branch February 13, 2024 00:26
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.

General Principles Page
5 participants