Skip to content
This repository has been archived by the owner on Dec 14, 2022. It is now read-only.

Web Platform #310

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Web Platform #310

wants to merge 4 commits into from

Conversation

hueyy
Copy link

@hueyy hueyy commented Sep 14, 2019

Expo for web is in beta and subject to breaking changes. Do not use this in production yet.

Patches

Not Working - To be fixed

  • SettingsScreen (because CheckBoxes)
  • StudySpacesDetailScreen (because Chart)
  • scrolling on TimetableScreen (because ViewPager)
  • RoomsDetailScreen (because Google Maps)
  • TimetableDetailScreen (because Google Maps)
  • JumpToDate popup on TimetableScreen

Aesthetic Issues

  • image sizing on SplashScreen (logo especially)
  • Floating action button has weird translucent rectangle behind it
  • Select element on EmptyRoomsScreen is unstyled

Misc non-critical issues

  • No analytics because the native Amplitude library doesn't work on web, so we need a web JS implementation
  • Ensure that Jest tests web as a platform as well (currently it only does iOS)

Resolves #104

@hueyy hueyy added enhancement New feature or request WIP work in progress labels Sep 14, 2019
@hueyy
Copy link
Author

hueyy commented Sep 15, 2020

@hueyy hueyy marked this pull request as draft September 15, 2020 04:34
@hueyy hueyy removed the WIP work in progress label Sep 15, 2020
@hueyy hueyy force-pushed the master branch 3 times, most recently from f77c0d1 to 4ca0b34 Compare July 5, 2021 06:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Look into React Native for Web
1 participant