Add wellness section with links and phone numbers V2 #151
Conversation
|
Thanks for the contribution! Looks great on a larger viewport, but there may be some problems integrating it into the mobile bottom navigation bar without significant modifications. I think it would be more appropriate to create a new tab in utilities dedicated to wellness; currently, there's a support tab which could work with your addition. Are there any specific resources that would be beneficial? I'd be interested to learn more. |
|
Thanks for the information Leo. Would it be possible to push the Crisis Support Tab up a level (ie. make it the same level as Courses and Staff so it can be more easily accessed by students)? |
|
Yup! I think that would be best too. I don't think the order of the tabs is too important after maybe the map or finals calc either. |
|
Ok, I will work on implementing that. |
aelnosu
changed the title
|
New Version |
|
Mobile test |
|
Also work on iPhone 5 |
|
|
|
Looking great! |
aelnosu
changed the title
|
This part of the code is ready for review. I will probably need to add another card, but that will be done when I have time™ |
|
Let me rebase this branch first |
This is only a placeholder for now. As most resources are not ready yet, I just added a dummy card.
Refactor crisis support resources: remove Support component and integrate its content into PhoneNumber component; update references in Resources and Wellness pages.
Update RED phone number to 988, also change reference from Phone1 to Phone
Added safety presenation by wellness as an image to the wellness page.
…tion Formalized all links and phone numbers in support cards
Added a new section "Support Others" with relevant resources and contact information.
|
Ok, all the changes are done :) |
33tm
left a comment
33tm
left a comment
There was a problem hiding this comment.
Looks good for the most part, I'd just check the sources and the phrasing describing them. Wouldn't include sources unless it has a direct relation to Gunn.
Additionally, all the anchors should have a target="_blank" to open in a new tab, preventing redirection away from WATT.
Co-authored-by: Leo Jeong <80350771+33tm@users.noreply.github.com>
Co-authored-by: Leo Jeong <80350771+33tm@users.noreply.github.com>
Co-authored-by: Leo Jeong <80350771+33tm@users.noreply.github.com>
Co-authored-by: Leo Jeong <80350771+33tm@users.noreply.github.com>
Co-authored-by: Leo Jeong <80350771+33tm@users.noreply.github.com>
Co-authored-by: Leo Jeong <80350771+33tm@users.noreply.github.com>
Co-authored-by: Leo Jeong <80350771+33tm@users.noreply.github.com>
…slides It just take too long to load one big image. Split it into multiple smaller images. Also used webp format for better compression.
Removed phone numebrs that are not officially endorsed by Gunn Wellness
Removed unendorsed external resources and added Care Solace as a recommended service.
… components Added className="break-all" to email & website links to ensure long addresses break correctly and do not overflow their containers. Also added _blank and rel="noopener noreferrer" to external links for security and usability.
IDK why it was there in the first place
Co-authored-by: Leo Jeong <80350771+33tm@users.noreply.github.com>
Co-authored-by: Leo Jeong <80350771+33tm@users.noreply.github.com>
Co-authored-by: Leo Jeong <80350771+33tm@users.noreply.github.com>
|
All changes are made and ready for review. If possible, could you merge it next week Wednesday? |
…rticleCard for presentation handling Use iframe for Presentation component and adjust ArticleCard to handle presentation prop, define size as a percentage of window size, not characters
|
|
|
Looks better for the most part, would use Tailwind breakpoints rather than creating a new I would double check with the official sources from the Gunn admin (??) you mentioned before since I see quite a bit of formatting inconsistencies. Could you send them over as well? Want to confirm that it's directly endorsed. Also some lines are really long, do you mind spreading some of your tags over multiple lines? Some are over 250 characters and are difficult to read. |
|
I think I found the sources you were using---they seem well formatted as is on the Gunn website; I'm thinking again and maybe this should be changed to a link to the Gunn website? I think there's a benefit of having a single source of truth for crisis resources, especially with how frequent and unpredictable Gunn website is. Maybe it's better to rethink this addition, what are your thoughts? |
|
I understand what you mean. The resources are on https://gunn.pausd.org/wellness/crisis-resources. The reason why this PR exists is to make them easily accessible. The information on Gunn's Website are not easily reachable form the main site (https://gunn.pausd.org/), and is just an information dump, hard to read and find what you need quickly. WATT could serve as a much more efficient way to obtain such information. |
2 participants
This PR adds a tab in the sidebar for wellness-related resources.