Skip to content

Comments

Give the next 5 open days#102

Merged
alifeee merged 7 commits intomainfrom
feature/timing
Aug 31, 2025
Merged

Give the next 5 open days#102
alifeee merged 7 commits intomainfrom
feature/timing

Conversation

@Rijndael1998
Copy link
Contributor

Closes #82

@Rijndael1998 Rijndael1998 requested a review from alifeee August 13, 2025 18:37
@Rijndael1998
Copy link
Contributor Author

image

@alifeee
Copy link
Member

alifeee commented Aug 13, 2025

ooo, I like it ! any chance of times ? 👉 👈

@Rijndael1998
Copy link
Contributor Author

ooo, I like it ! any chance of times ? 👉 👈

Of course, i'm thinking the design could be better too. What do you think?

@alifeee
Copy link
Member

alifeee commented Aug 14, 2025

Of course, i'm thinking the design could be better too. What do you think?

how's something like this?

image
<div id="next-days" style="margin-top: 1em; margin-bottom: 1em;"><h3 style="margin: 0px; padding: 0px;">Next few nights</h3><ul><li><span class="in">(today!)</span> 6–9pm, Thursday 14<sup>th</sup> August</li><li><span class="in">(in 2 days)</span> 2–6pm, Saturday 16<sup>th</sup> August</li><li><span class="in">(in 4 days)</span> 6–9pm, Monday 18<sup>th</sup> August</li><li><span class="in">(in 6 days)</span> 6–9pm, Wednesday 20<sup>th</sup> August</li><li><span class="in">(in 7 days)</span> 6–9pm, Thursday 21<sup>st</sup> August</li><li></li></ul></div>
#next-days .in {
  display:inline-block;
  width: 6rem;
  opacity: 0.5;
}

@Rijndael1998
Copy link
Contributor Author

I can do that

@Rijndael1998
Copy link
Contributor Author

Rijndael1998 commented Aug 18, 2025

image

Is this good enough? @alifeee

@Rijndael1998
Copy link
Contributor Author

Just don't merge yet, i want to clean up the code once you like how it looks

@alifeee
Copy link
Member

alifeee commented Aug 19, 2025

looks pretty nice I think :]

@Rijndael1998
Copy link
Contributor Author

looks pretty nice I think :]

I'll clean up then c:

@Rijndael1998
Copy link
Contributor Author

@alifeee
image

@Rijndael1998
Copy link
Contributor Author

If you're ok with the changes, you can merge it now.

@alifeee
Copy link
Member

alifeee commented Aug 26, 2025

hmmm

image

@alifeee
Copy link
Member

alifeee commented Aug 26, 2025

I like it apart from the undefined. Though, I think the layout of the site is… weird which is accentuated by this change

because… the calendar appears on 3 separate pages. and now that it's quite big, it is a bit weird to be duplicated so much.

really, the website could probably just be a single page homepage and a blog

but that's not related to this issue

@Rijndael1998
Copy link
Contributor Author

Rijndael1998 commented Aug 28, 2025

Sorry @alifeee, it's been fixed now. It's really good that you tested this again because it was working for me 3 days ago 😅

I won't be able to make more code changes to the website for a while. Once this PR completes, I'm going on hiatus.

@Rijndael1998 Rijndael1998 requested review from alifeee and removed request for alifeee August 28, 2025 21:47
Copy link
Member

@alifeee alifeee left a comment

Choose a reason for hiding this comment

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

looks good :]

I expect many pages will be coalesced into one page sometime soon, so all the repetition that is currently on the website will be less

@alifeee alifeee merged commit 337f9f5 into main Aug 31, 2025
1 check passed
@alifeee alifeee deleted the feature/timing branch September 2, 2025 17:31
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.

2 participants