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

Fix frontend issues on resources page #103

Merged
merged 10 commits into from
Oct 31, 2023
Merged

Conversation

samderanova
Copy link
Contributor

@samderanova samderanova commented Oct 30, 2023

  • Set preserveAspectRatio=none for the backend resource description index card to resolve text overflow issue
  • Created a new ResourceCard component that combines the API and backend resource card components into one to allow simpler usage and scalability
    • Removed assets that became unused as a result

@samderanova samderanova linked an issue Oct 30, 2023 that may be closed by this pull request
@github-actions
Copy link

github-actions bot commented Oct 30, 2023

Deploy preview for zothacks-site-2023 ready!

Name Hack at UCI Site
Preview Visit Preview
Commit f8133c2

@github-actions
Copy link

github-actions bot commented Oct 30, 2023

Deploy preview for zothacks-site-2023-sanity ready!

Name Sanity Studio
Preview Visit Preview
Commit f8133c2

@samderanova samderanova requested review from ryanluu12345, alexanderl19 and tyleryy and removed request for ryanluu12345 October 30, 2023 08:18
@tyleryy
Copy link
Contributor

tyleryy commented Oct 31, 2023

The new window icon looks a bit off. It looks misaligned with the text. tbh maybe just use the whole tag image if it is too hard to do due to deadlines.

Screen Shot 2023-10-30 at 5 19 37 PM

@tyleryy
Copy link
Contributor

tyleryy commented Oct 31, 2023

If we need to merge fast, i'll approve without any changes.

Copy link
Member

@taesungh taesungh left a comment

Choose a reason for hiding this comment

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

Although I miss the angled clear tape pieces, this approach does provide more flexibility in layout choices and eliminates the duplication between what used to be two separate resource card components. Minor notes otherwise.

tyleryy
tyleryy previously approved these changes Oct 31, 2023
- Include `translateZ(0)` when applying transform to drop shadow filter
Copy link
Contributor

@tyleryy tyleryy left a comment

Choose a reason for hiding this comment

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

Lgtm

@samderanova samderanova merged commit e66f4ad into main Oct 31, 2023
2 checks passed
tyleryy pushed a commit that referenced this pull request Nov 3, 2023
* update: created getSchedule file

* update: synced sanity data with schedule component

* feat: update event schema

* feat: group events by day

* feat: format date ranges

* fix: group events by America/Los_Angeles day

* fix: event order asc

* fix: remove resources from nav

---------

Co-authored-by: Tyler Yu <tyleryy@uci.edu>

Fix frontend issues on resources page (#103)

* fix: backend description text overflow

* feat: new ResourceCard component

removed unused assets

* fix: key prop error

* fix: remove unused components

* fix: references to unused images

* refactor: short-circuit and color fixes

* Fix transformed filter glitches for webkit

- Include `translateZ(0)` when applying transform to drop shadow filter

---------

Co-authored-by: Taesung Hwang <taesungh@uci.edu>

fix: title and location mobile alignment (#109)

feat: frontend resources section

feat: starter packs section

removed placeholder config files

feat: add day of week to schedule (#110)

* feat: add day of week to schedule

* fix: replace weekday list with date format

* fix: long month and date format

update: restore resources link in navbar

fix: timezone inconsistencies (#119)

* fix: timezone inconsistencies

* fix: timezone inconsistencies

fix: backend resources grid issue

feature: clipboard paper animation (#116)

undefined

Feature/clipboard animation update (#123)

* feature: clipboard paper animation

* fix: padding adjustment on date

* fix: border radius and page animation update

feat: incident response form

fix: remove open in new window icon

Setup animated countdown on schedule page (#93)

* setup: basic framer countdown

* fix: removed useRef, changed animation exit value

* update: temporarily disabled slide up animation

* fix: conditional day render

* fix: precompute time

* fix: margin for the countdown

* update: added loading bar

* fix: rename to Loader

* feat: time until hacking countdown timer

* fix: clipboard styling

---------

Co-authored-by: Tyler Yu <tyleryy@uci.edu>
Co-authored-by: Alexander Liu <a@alexanderliu.com>

fix: match time zones on server and client

fix: missing date-fns-tz import (#126)

fix: disable page scroll x (#125)

refactor: move homepage from views folder to app folder

refactor: convert components to arrow functions and reexport

feat: devpost

fix: changed import path

fix: added ease and moved component
samderanova pushed a commit that referenced this pull request Nov 3, 2023
* feat: svg animation rough draft

* update: increase stroke width

* update: new svg

* update: added timing to opacity

* update: switched to animate to create stagger

* fix: adjusted timings

* fix: timing

* fix: more timing adjustments

* fix: type issues with initial variant

* fix: responsive svg

* fix: removed old title svg

* fix: added useEffect dep

* fix: remove unused imports

* fix: adjusted clamp

* update: refactored animated title

fix: visually hidden title

update: refactored animated title

Use lined paper vector and remove unnecessary nav (#62)

* fix: use lined paper vector and remove unnecessary nav

* fix: stick nav bar to top of screen

* fix: remove sticky navbar for mobile

---------

Co-authored-by: Sam Der <sder@uci.edu>

fix: change resources PNGs to SVGs

hotfix: heart and star sticker positioning

Add ZotHacks Open Graph Properties (#71)

* feat: add OG image

* fix: use opengraph-image for OG generation

Create Sanity resources schema (#91)

* feat: resource schema

* feat: icon, preview, and color field description

update: TikTok logo and improved alt text (#83)

* update: TikTok logo and improved alt text

* fix: switch to svg version

fix: use container spacing for resource page (#79)

* fix: use container spacing for resource page

* temp: fix row/col setup

* remove col div

update: remove apply links and mentor section (#89)

* update: remove apply links and mentor section

* fix: restore apply button, adjust wording

* fix: disabled mentor apply button

* update: zothacks 2023

* fix: removed extra import

* fix: added variant type

* fix: changed import path

* fix: added ease and moved component

* Feature: SVG ZotHacks Title animation

* update: created getSchedule file

* update: synced sanity data with schedule component

* feat: update event schema

* feat: group events by day

* feat: format date ranges

* fix: group events by America/Los_Angeles day

* fix: event order asc

* fix: remove resources from nav

---------

Co-authored-by: Tyler Yu <tyleryy@uci.edu>

Fix frontend issues on resources page (#103)

* fix: backend description text overflow

* feat: new ResourceCard component

removed unused assets

* fix: key prop error

* fix: remove unused components

* fix: references to unused images

* refactor: short-circuit and color fixes

* Fix transformed filter glitches for webkit

- Include `translateZ(0)` when applying transform to drop shadow filter

---------

Co-authored-by: Taesung Hwang <taesungh@uci.edu>

fix: title and location mobile alignment (#109)

feat: frontend resources section

feat: starter packs section

removed placeholder config files

feat: add day of week to schedule (#110)

* feat: add day of week to schedule

* fix: replace weekday list with date format

* fix: long month and date format

update: restore resources link in navbar

fix: timezone inconsistencies (#119)

* fix: timezone inconsistencies

* fix: timezone inconsistencies

fix: backend resources grid issue

feature: clipboard paper animation (#116)

undefined

Feature/clipboard animation update (#123)

* feature: clipboard paper animation

* fix: padding adjustment on date

* fix: border radius and page animation update

feat: incident response form

fix: remove open in new window icon

Setup animated countdown on schedule page (#93)

* setup: basic framer countdown

* fix: removed useRef, changed animation exit value

* update: temporarily disabled slide up animation

* fix: conditional day render

* fix: precompute time

* fix: margin for the countdown

* update: added loading bar

* fix: rename to Loader

* feat: time until hacking countdown timer

* fix: clipboard styling

---------

Co-authored-by: Tyler Yu <tyleryy@uci.edu>
Co-authored-by: Alexander Liu <a@alexanderliu.com>

fix: match time zones on server and client

fix: missing date-fns-tz import (#126)

fix: disable page scroll x (#125)

refactor: move homepage from views folder to app folder

refactor: convert components to arrow functions and reexport

feat: devpost

fix: changed import path

fix: added ease and moved component

* fix: unused asset

---------

Co-authored-by: Alexander Liu <a@alexanderliu.com>
@taesungh taesungh deleted the fix/resources-frontend-issues branch September 27, 2024 02:24
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.

Resources Page: Resolve Frontend Issues
3 participants