Added the Calendar View of all the events #96
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
components/Header/Calendar.jsx
Outdated
| </iframe> | ||
| </div> | ||
| ) | ||
| } |
There was a problem hiding this comment.
This little icon means you need to add an empty line at the end of everything
components/Header/Header.jsx
Outdated
| return ( | ||
| <div> | ||
| <Navigation /> | ||
| </div> |
There was a problem hiding this comment.
Why do we need to add a div here?
There was a problem hiding this comment.
Agreed, we could remove this
package-lock.json
Outdated
There was a problem hiding this comment.
We shouldn't include changes in this file, unless we added something to the package.json
components/Header/Header.jsx
Outdated
| return ( | ||
| <div> | ||
| <Navigation /> | ||
| </div> |
There was a problem hiding this comment.
Agreed, we could remove this
| @@ -36,7 +43,14 @@ function classNames(...classes) { | |||
| } | |||
|
|
|||
| export default function Navigation() { | |||
| const [showCalendar, setShowCalendar] = useState(false); | |||
|
|
|||
| const handleCalendarClick = () => { | |||
There was a problem hiding this comment.
@daynamariejpembie sorry if I wasn't explicit on this.
But basically we only need here is a new Page with the calendar, where the users could go back and forward from it.
Lets create a new Page and call it Calendar, add the Iframe there, then we should be good to go there.
There was a problem hiding this comment.
@jonathans199 I'm taking a look at this to help @daynamariejpembie. To be clear, what you're talking about doing is making a calendar.js page under the pages directory right? And that would just contain the iframe. Then there needs to be ternary (or some other way of bringing the calendar to view) inside components/Header/Navigation.jsx that disables the mobile or medium and up sized buttons and then renders the calendar? The second part is tripping me up in particular, and that may not be how it needs to rendered. Please elaborate so we can implement this, thanks!
I have added the Calendar view of the tech alliance events. This is for issue #52
Below are screenshots of what I achieved, any feedback is much appreciated: