Skip to content

[BUGFIX] adjust image height settings for news cards template#382

Merged
aennafae merged 1 commit intot3kit:masterfrom
dkd:news-cards-image-height
Jan 30, 2018
Merged

[BUGFIX] adjust image height settings for news cards template#382
aennafae merged 1 commit intot3kit:masterfrom
dkd:news-cards-image-height

Conversation

@aennafae
Copy link
Contributor

@aennafae aennafae commented Jan 30, 2018

Background image and dummy image should always have same height to avoid layout break.
Image height is set to 768px to fill column width in tablet viewport.

@aennafae aennafae merged commit 8215842 into t3kit:master Jan 30, 2018
@aennafae aennafae deleted the news-cards-image-height branch January 30, 2018 14:14
@dmh
Copy link
Member

dmh commented Feb 15, 2018

Will be reverted because looks like this is not a proper fix for this #351 issue. Tested on t3kit default site. Images still wrongly scaled. Should be reviewed again.

dkd-oertel added a commit to dkd/theme_t3kit that referenced this pull request Mar 22, 2018
dmh pushed a commit that referenced this pull request Jun 7, 2018
…mages. Accessibility

* [FEATURE] revert "[BUGFIX] partly revert - improve news list accessibility (#343)"

This reverts commit ed1b624.

* [FEATURE] revert "[BUGFIX] revert - change markup for news list (#373)"

This reverts commit a4286d0.

* [FEATURE] revert "[BUGFIX] revert PR adjust image height settings for news cards template (#382)"

This reverts commit b305f6a.

* [FEATURE] revert "[BUGFIX] revert PR Improve news cards accessibility (#351)"

This reverts commit 2d33074.

* [FEATURE] revert "[BUGFIX] revert news-carousel accessibility improvements (#352)"

This reverts commit f1b155e.

* Issue #83227: Add object-fit polyfill and remove background images

* update css/less

* remove unnecessary tag to fix error

* adjust less formatting for stylelint

* update css/less

* remove wrong indentation

* update css/less
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