Updated Export Page #63
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
amayaevan
commented
Mar 29, 2020
amayaevan
commented
- Updated Export page with better visuals, toast notifications, anima…
- Changed return type for CSV and PDF in order to bubble up success/fail to UI
- Added GetFlutter UI Widget library
…tions - Changed return type for CSV and PDF in order to bubble up success to UI - Added GetFlutter UI Widget library
PixelDotPixel
approved these changes
Mar 30, 2020
Changed background Changed sidebar Image to OSU Logo, looks better and more professional.
|
Toggle between the different background images while interacting with the UI to get a full evaluation of what you guys find best. Change line 48 in Export/index to the other image in assets/images. I also downloaded the HQ OSU logos from the OSU website and tried a better image in the Sidebar nav. Overall I think it looks more clear and simple. Ideally we could also put the full civil/construction logo as an initial landing image at the bottom for our Project Page. Or possibly on the splash screen for loading the application. |
jctrotter
requested changes
Mar 31, 2020
added flex to widgets for different screens.
jctrotter
requested changes
Mar 31, 2020
…fter sending email changed font weight to be more noticeable on switch for the time being
jctrotter
approved these changes
Apr 1, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.