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

Adding 'Identity' CSS for missing event image #387

Merged
merged 1 commit into from
Dec 15, 2023

Conversation

kasperbirch1
Copy link
Contributor

@kasperbirch1 kasperbirch1 commented Dec 12, 2023

Link to issue

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

Description

This pull request introduces the "identity" CSS class to manage scenarios involving missing images.

  • Implementation of the %identity-placeholder class, which sets the background color and adds a bold border.
  • Introduction of the image-credited__no-image class, an extension of %identity-placeholder.
  • Addition of the height: auto property to img elements, ensuring the aspect ratio of images is maintained.

Screenshot of the result

image

@rasben
Copy link
Contributor

rasben commented Dec 14, 2023

Please look at the UI tests before assigning reviewers :)

@JacobArrow JacobArrow closed this Dec 15, 2023
@JacobArrow JacobArrow reopened this Dec 15, 2023
- Implement `%identity-placeholder` to set the background color and apply a bold border.
- Introduce `image-credited__no-image` class that extends `%identity-placeholder`.
@kasperbirch1 kasperbirch1 force-pushed the DDFFORM-75-hovedbillede-til-arrangement branch from 9d75a4e to ffd072b Compare December 15, 2023 13:30
@kasperbirch1 kasperbirch1 assigned rasben and unassigned kasperbirch1 Dec 15, 2023
@kasperbirch1 kasperbirch1 requested a review from rasben December 15, 2023 13:34
@rasben rasben merged commit 13c6838 into release/2023-51-0 Dec 15, 2023
9 checks passed
@rasben rasben deleted the DDFFORM-75-hovedbillede-til-arrangement branch December 15, 2023 13:35
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