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.

Sign up for GitHub

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

Jump to bottom

Patrick dev changes #30

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

Patrick dev changes #30

wants to merge 41 commits into from

Conversation

McCorsa
Copy link
Collaborator

@McCorsa McCorsa commented Mar 9, 2024

No description provided.

Patadam added 30 commits January 31, 2024 14:49
@Patadam
build(update): dependency version bump to latest
863636b
@Patadam
all!: BREAKING Major overhaul
8ce8fc6
@Patadam
feat(events): add events information page
6ca4be6
@Patadam
fix(event): add url as prop
81ff3c0
@Patadam
feat(events): add events information to home page
6a5aa22
@Patadam
fix(events): limit max shown events to 5
39ff46c
@Patadam
fix(events): add responsive layout
046c2f4
@Patadam
refactor: remove deprecated resources
269fd08
@Patadam
feat(readme): provide information on creating pages and changing data
2b81549
@Patadam
fix(imports): fix page and resource imports
7f5ca43
@Patadam
refactor(rules): create new rules page
d471c83
@Patadam
refactor: remove old rules page
b2d99f6
@Patadam
fix(title): allow passing title prop to page to set web title
d0fb6b5
@Patadam
refactor(AGM): reformat AGM
dccf4a0
@Patadam
refactor(rules): replace rules with updated component format
80b8aa7
@Patadam
fix(title): update title to exclude | code network
91dc369
@Patadam
fix(title): auto append | code network to title
7b3378c
@Patadam
fix(header): make navbar mobile responsive.
5139222
@Patadam
fix(mobile): add mobile responsivity
cb8cac9
@Patadam
fix: remove
2726b24
@Patadam
fix(sponsors): show all sponsor levels
3d5415b
@Patadam
fix: show label when sponsor level exists
9bc95e3
@Patadam
update: update for 2024 sponsors
fab4467
@Patadam
fix(header): stick to top of screen
22ffb8e
@Patadam
fix(events): handle no event state
b5ced06
@Patadam
fix: add ;
76d6cce
@Patadam
refactor: move rules into tabs
6976e56
@Patadam
fix: hide events until complete
73f9fa5
@Patadam
dev: move events to new branch
9a5ef17
@Patadam
fix: remove unused link
0bcfbcc
@netlify Netlify
Copy link

netlify bot commented Mar 9, 2024 •
edited
Loading

✅ Deploy Preview for radiant-truffle-7640cb ready!

Name Link
🔨 Latest commit 4c4b314
🔍 Latest deploy log https://app.netlify.com/sites/radiant-truffle-7640cb/deploys/65ffa08395d53d00082cac87
😎 Deploy Preview https://deploy-preview-30--radiant-truffle-7640cb.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@McCorsa
Copy link
Collaborator Author

McCorsa commented Mar 9, 2024

@Patadam A few issues I can see at the moment on the deploy preview:

  • Navigation items are looking quite crammed together on the top-right corner
  • The CN icon in the navbar is off-centred. I imagine this is to do with the navigation links on the right taking up more space, but I'm sure there is a workaround for this.
  • The CN icon shrinks quite dramatically when I resize the window:
    image
  • The background graphic on homepage is in an interesting position. I feel like it should somehow be anchored to the right-hand side of the screen, not the middle.

Otherwise, off to a great start with the refactoring. Keep it up!

@Patadam Patadam marked this pull request as draft June 16, 2024 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@McCorsa @Patadam