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

Add EventPage + EventHeader #353

Merged
merged 8 commits into from
Dec 5, 2023

Conversation

kasperbirch1
Copy link
Contributor

Link to issue

https://reload.atlassian.net/browse/DDFFORM-14

Description

This pull request introduces two new components: EventPage and EventHeader. These components are used for the event-related pages of our application.

Screenshot of the result

image image

@kasperbirch1 kasperbirch1 force-pushed the DDFFORM-14-oprette-arrangement branch from 909b05c to 834c667 Compare November 29, 2023 14:15
@kasperbirch1 kasperbirch1 force-pushed the DDFFORM-14-oprette-arrangement branch 2 times, most recently from 092a3fb to 5170a07 Compare November 30, 2023 13:27
kasperbirch1 added a commit to danskernesdigitalebibliotek/dpl-cms that referenced this pull request Nov 30, 2023
This should be reverted when the pull requests in dpl-design-system are merged.
danskernesdigitalebibliotek/dpl-design-system#353
kasperbirch1 added a commit to danskernesdigitalebibliotek/dpl-cms that referenced this pull request Nov 30, 2023
This should be reverted when the pull requests in dpl-design-system are merged.
danskernesdigitalebibliotek/dpl-design-system#353
kasperbirch1 added a commit to danskernesdigitalebibliotek/dpl-cms that referenced this pull request Nov 30, 2023
This should be reverted when the pull requests in dpl-design-system are merged.
danskernesdigitalebibliotek/dpl-design-system#353
@kasperbirch1 kasperbirch1 force-pushed the DDFFORM-14-oprette-arrangement branch from 5170a07 to aed6a2b Compare November 30, 2023 14:22
kasperbirch1 added a commit to danskernesdigitalebibliotek/dpl-cms that referenced this pull request Dec 1, 2023
This should be reverted when the pull requests in dpl-design-system are merged.
danskernesdigitalebibliotek/dpl-design-system#353
Copy link
Contributor

@kasperg kasperg left a comment

Choose a reason for hiding this comment

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

👍 Good stuff here.

I appreciate you doing an effort to use the right HTML elements. I think we can do even better.

src/stories/Library/event-header/EventHeader.tsx Outdated Show resolved Hide resolved
src/styles/scss/container.scss Show resolved Hide resolved
src/stories/Library/event-header/EventHeader.tsx Outdated Show resolved Hide resolved
src/stories/Library/event-header/EventHeader.tsx Outdated Show resolved Hide resolved
src/stories/Library/event-header/EventHeader.tsx Outdated Show resolved Hide resolved
src/stories/Library/event-header/EventHeader.tsx Outdated Show resolved Hide resolved
src/stories/Library/event-header/EventHeader.tsx Outdated Show resolved Hide resolved
src/stories/Blocks/event-page/EventPage.tsx Outdated Show resolved Hide resolved
@kasperg kasperg removed their assignment Dec 1, 2023
@kasperbirch1 kasperbirch1 force-pushed the DDFFORM-14-oprette-arrangement branch 2 times, most recently from c6f529e to 305f262 Compare December 3, 2023 12:57
@kasperbirch1 kasperbirch1 requested a review from kasperg December 3, 2023 13:01
kasperbirch1 added a commit to danskernesdigitalebibliotek/dpl-cms that referenced this pull request Dec 3, 2023
This should be reverted when the pull requests in dpl-design-system are merged.
danskernesdigitalebibliotek/dpl-design-system#353
Copy link
Contributor

@kasperg kasperg left a comment

Choose a reason for hiding this comment

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

👍 I like the new changes.

The only thing I that needs to be addressed now is the usage of <figure>.

src/stories/Library/event-header/EventHeader.tsx Outdated Show resolved Hide resolved
src/stories/Library/image-credited/ImageCredited.tsx Outdated Show resolved Hide resolved
Comment on lines +22 to +29
.event-header__content {
display: grid;
grid-template-columns: $s-3xl 1fr $s-3xl;
grid-template-areas:
". tag ."
". date ."
". title ."
"btn btn btn";
Copy link
Contributor

Choose a reason for hiding this comment

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

As I read this you are creating padding for some areas using columns. That's creative.

Implemented a new SCSS mixin named 'container' to handle different container sizes: small, medium, and large. This mixin allows for dynamic styling based on the specified size parameter. Styles for 'small' and 'large' sizes are structured for future customization, while the 'medium' size is currently set with a max-width of 1440px and centered margin.
Introduced markup and CSS for `EventHeader`, now included in `EventPage`.

Additionally, made small modifications to the `tag` component to allow the addition of extra CSS classes. In typography, I introduced a placeholder for `.text-header-h1` to facilitate future extension of this class
Enabling user experimentation with various images.
CSS handles uppercase transformation
-Removed the `className` property from the `Button` component due to redundancy.
- Planned additional refactoring to extend the "button" classes within `event-header__button`
Extracted into a separate component to facilitate easy integration in diverse application areas.
…as` for content positioning.

Removed unnecessary wrapper div in `.event-header__content` for better semantic structure
@kasperbirch1 kasperbirch1 force-pushed the DDFFORM-14-oprette-arrangement branch from 305f262 to 2c7aa0a Compare December 4, 2023 10:50
@kasperbirch1 kasperbirch1 requested a review from kasperg December 4, 2023 10:51
kasperg
kasperg previously approved these changes Dec 5, 2023
Copy link
Contributor

@kasperg kasperg left a comment

Choose a reason for hiding this comment

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

👍 LGTM

@kasperg kasperg dismissed their stale review December 5, 2023 08:25

We need to merge this into 2023-50-0

@kasperg kasperg changed the base branch from release/2023-49-0 to release/2023-51-0 December 5, 2023 12:41
Copy link
Contributor

@kasperg kasperg left a comment

Choose a reason for hiding this comment

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

👍 LGTM

@kasperbirch1 kasperbirch1 merged commit 00ec24b into release/2023-51-0 Dec 5, 2023
9 checks passed
@kasperbirch1 kasperbirch1 deleted the DDFFORM-14-oprette-arrangement branch December 5, 2023 13:10
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.

3 participants