Skip to content

Corrected wordwrap long titles latest articles#8318

Closed
pe7er wants to merge 1 commit intojoomla:stagingfrom
pe7er:module-latest-articles
Closed

Corrected wordwrap long titles latest articles#8318
pe7er wants to merge 1 commit intojoomla:stagingfrom
pe7er:module-latest-articles

Conversation

@pe7er
Copy link
Copy Markdown
Contributor

@pe7er pe7er commented Nov 7, 2015

This PR fixes the layout of long titles in the Latest Articles Module of the Protostar template
Thanks @waader for spotting this issue!

Testing Instructions

See similar issue: #8312

Before the PR

mod_latestnews-before

After the PR

mod_latestnews-after

@waader
Copy link
Copy Markdown
Contributor

waader commented Nov 7, 2015

I have tested this item ✅ successfully on 83029b6

Thanks pe7er!


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

@w13ear
Copy link
Copy Markdown

w13ear commented Dec 12, 2015

I have tested this item ✅ successfully on 83029b6

Test was succesfull.
I have made an article with a lang headerand long content.
Results before and after the patch were as aspected.


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

@brianteeman
Copy link
Copy Markdown
Contributor

Based on @Bakual comments I am setting this to Needs Review for a maintainer to make a decision


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

@roland-d
Copy link
Copy Markdown
Contributor

roland-d commented May 7, 2016

@pe7er Can you please fix the merge conflicts? Thanks.

@pe7er
Copy link
Copy Markdown
Contributor Author

pe7er commented May 7, 2016

This PR can be closed because of new PR #10272

@brianteeman
Copy link
Copy Markdown
Contributor

Closed as requested

@brianteeman brianteeman closed this May 7, 2016
@pe7er pe7er deleted the module-latest-articles branch September 25, 2018 13:45
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.

6 participants