-
Notifications
You must be signed in to change notification settings - Fork 3
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
created an accordion style dropdown for each round #104
Conversation
<h2> {{ $round->name }} </h2> | ||
<div class="scrollable"> | ||
<h2 class="appTitle"> {{ $round->name }} </h2> | ||
<div class="scrollable hidden"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Meleeman01 As per the requirements in the original issue #63, only rounds that are older than a year should have their applications hidden by default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
aight I got that in there. I used Carbon's isLastYear() function and it seems to do it only if the date is from last year and not if the date is exactly 1 year. not sure which one you wanted but I can change it if you like.
4ebe801
to
f6adeb1
Compare
f6adeb1
to
19af682
Compare
|
||
<!--check to see if the round end_date is a year old from today--> | ||
|
||
@if(Carbon::parse($round->end_date)->diffInDays(Carbon::now())==365) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Um, wouldn't this only work if the difference in days is exactly 365?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
interestingly enough it works regardless when using diffInYears(), but i think thats something that could be made more clear as well. I tested both and they work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works as expected. My only suggestion would be to use the cursor: pointer
CSS property so users know that the headings can be clicked on.
No description provided.