? NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar drmenzelit
drmenzelit
15 Dec 2020

Pull Request for Issue #31647 .

Summary of Changes

Added height: auto to img to maintain aspect ratio when width 100%

Testing Instructions

Create an article with a full image in format 16:9 (e.g. 1280 x 720px), set Image Class to float-none.
Install this PR and run "npm run build:css"

Actual result BEFORE applying this Pull Request

The image will have a width of 100%, but the height stay at 720px, so the aspect ratio is distorted.

Expected result AFTER applying this Pull Request

The image is 100% wide and the height scale proportionally.

avatar drmenzelit drmenzelit - open - 15 Dec 2020
avatar drmenzelit drmenzelit - change - 15 Dec 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Dec 2020
Category Front End Templates (site) NPM Change
avatar richard67
richard67 - comment - 15 Dec 2020

@drmenzelit The phpcs failure in Drone is not related to your PR, it comes from the 4.0-dev branch having had code style errors when you had created your branch based on it. But if you want to get rid of the Drone failure, update your branch to latest 4.0-dev, where the errors meanwhile have been corrected.

avatar drmenzelit drmenzelit - change - 15 Dec 2020
Labels Added: ? NPM Resource Changed ?
avatar chmst
chmst - comment - 16 Dec 2020

I have tested this item successfully on 9737eba


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31667.

avatar chmst chmst - test_item - 16 Dec 2020 - Tested successfully
avatar gostn
gostn - comment - 17 Dec 2020

I have tested this item successfully on 9737eba


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31667.

avatar gostn gostn - test_item - 17 Dec 2020 - Tested successfully
avatar infograf768 infograf768 - change - 17 Dec 2020
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 17 Dec 2020

rtc


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31667.

avatar wilsonge wilsonge - close - 17 Dec 2020
avatar wilsonge wilsonge - merge - 17 Dec 2020
avatar wilsonge wilsonge - change - 17 Dec 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-12-17 15:17:57
Closed_By wilsonge
Labels Added: ?
avatar wilsonge
wilsonge - comment - 17 Dec 2020

Thanks!

Add a Comment

Login with GitHub to post a comment