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

Minor design tweaks #405

Merged
merged 10 commits into from
Mar 27, 2022
Merged

Minor design tweaks #405

merged 10 commits into from
Mar 27, 2022

Conversation

ghost
Copy link

@ghost ghost commented Mar 27, 2022

PROPOSAL

Minor tweaks to components I haven't gotten to yet. Also to make them comply with the new CSS.

CHANGELOG

  • Corrected text sizing for paragraphs
  • Depreciated old connect page

This is change #6 of my big update to prepare for next semester's codebase.

fixes #398

- Responsive text sizing via clamp()
- Transitioning CSS macros to new naming scheme
- Introduced --perma-light, --card-bg, --button-bg, and --button-hover variables for dynamic theming
@ghost ghost added this to the v3.0 🌞 milestone Mar 27, 2022
@ghost ghost requested a review from EthanThatOneKid March 27, 2022 03:14
@vercel
Copy link

vercel bot commented Mar 27, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/ethanthatonekid/acm-csuf-site/Fe1ojH5kWakwW5q8Kz6pexJC49Cu
✅ Preview: https://acm-csuf-site-git-mike-design-tweaks-ethanthatonekid.vercel.app

@jaasonw jaasonw mentioned this pull request Mar 27, 2022
mikeploythai added 6 commits March 27, 2022 15:21
- Changed font macros for paragraphs from size-xs to size-sm
- Images use clamp() to scale dynamically
- AcmButton re-written to use --button-bg and --button-hover for different styles in light/dark themes
- Modified wording in some paragraphs
- Removed what-is-acm.svelte, introduced about-us.svelte
- Modified about/index.svelte to accomodate for about-us.svelte
- Updated board member list
- officer-profile-list.svelte defaults grid to 4 diamonds instead of 3 in desktop mode
- Modified officers.json to be alphabetical
- Modified tiers.json for role placement
- Cards scale up on hover & active state
- Shadows get softer as the card becomes bigger
- Outline defaults to ACM color in initial state to be less contrasty to the shadow
- Join button hover effect consistent with other ACM buttons now
- Also deprecated old connect page
@jaasonw jaasonw force-pushed the mike/design-tweaks branch from 99b0589 to 9f578e1 Compare March 27, 2022 22:21
@vercel vercel bot temporarily deployed to Preview March 27, 2022 22:21 Inactive
@jaasonw
Copy link
Contributor

jaasonw commented Mar 27, 2022

Removing navbar and footer changes from this pr bcus they're in #401

Copy link
Contributor

@jaasonw jaasonw left a comment

Choose a reason for hiding this comment

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

Rest of the design tweaks look good to me

@vercel vercel bot temporarily deployed to Preview March 27, 2022 22:38 Inactive
Copy link
Contributor

@jaasonw jaasonw left a comment

Choose a reason for hiding this comment

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

image

jk i am no longer ok with these changes as it fails a check i do not know how to fix

@vercel vercel bot temporarily deployed to Preview March 27, 2022 22:51 Inactive
Copy link
Contributor

@jaasonw jaasonw left a comment

Choose a reason for hiding this comment

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

jk i fixed it

@jaasonw jaasonw merged commit e8d303a into main Mar 27, 2022
@ghost ghost deleted the mike/design-tweaks branch April 17, 2022 17:13
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.

pretty sure president should take precedence over event coordinator here
1 participant