NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar Harmageddon
Harmageddon
6 Feb 2021

Pull Request for Issue #32296.

This is an alternative solution to #32322. You'll need NPM to test this PR!

Summary of Changes

Added an overflow:hidden so that blog item divs don't expand with <pre> tags inside their content.

Testing Instructions

  1. Create an article. In the content, add a HTML <pre> tag with at least one very long line inside it.
  2. Mark the article as featured and add it to a category.
  3. Look at the article in frontend in the following views:
  • single article
  • category blog
  • featured blog

Actual result BEFORE applying this Pull Request

The pre tag destroys the blog layouts (but is correctly displayed in single article view).

Screenshot_2021-02-06 actual

Expected result AFTER applying this Pull Request

The pre tag gets a vertical scrollbar and is displayed inside the limits of the article.

Screenshot_2021-02-06 expected

Documentation Changes Required

None

avatar Harmageddon Harmageddon - open - 6 Feb 2021
avatar Harmageddon Harmageddon - change - 6 Feb 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Feb 2021
Category Front End Templates (site) NPM Change
avatar ceford ceford - test_item - 11 Feb 2021 - Tested successfully
avatar ceford
ceford - comment - 11 Feb 2021

I have tested this item successfully on 1a40e91

The pre line does not obviously scroll and is only recognisable from the font difference - would it benefit from some additional style? I guess authors could apply their own class.


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

avatar ChristineWk ChristineWk - test_item - 13 Feb 2021 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 13 Feb 2021

I have tested this item successfully on 1a40e91

Used Prebuilt Package


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

avatar richard67 richard67 - change - 13 Feb 2021
Status Pending Ready to Commit
Labels Added: ? NPM Resource Changed
avatar richard67
richard67 - comment - 13 Feb 2021

RTC


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

avatar wilsonge wilsonge - change - 17 Feb 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-02-17 18:56:45
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 17 Feb 2021
avatar wilsonge wilsonge - merge - 17 Feb 2021
avatar wilsonge
wilsonge - comment - 17 Feb 2021

Thanks!

Add a Comment

Login with GitHub to post a comment