Skip to content

Fixed long title in Smart Search: Indexed Content#8316

Merged
roland-d merged 1 commit intojoomla:stagingfrom
pe7er:com_finder-long-title-wordwrap
Nov 10, 2015
Merged

Fixed long title in Smart Search: Indexed Content#8316
roland-d merged 1 commit intojoomla:stagingfrom
pe7er:com_finder-long-title-wordwrap

Conversation

@pe7er
Copy link
Copy Markdown
Contributor

@pe7er pe7er commented Nov 7, 2015

This PR fixes the layout for long titles in Smart Search: Indexed Content in the Isis template

Testing Instructions

Before the PR

Create a new article with a long title as described in #8312
Go to Components > Smart Search > Indexed Content
to see that the long title messes up the layout.

com_finder-longtitle

After the PR

This PR should fix the layout

com_finder-longtitle-after

@waader
Copy link
Copy Markdown
Contributor

waader commented Nov 7, 2015

I have tested this item ✅ successfully on 8a7d35c


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

@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Nov 7, 2015

I have tested this item ✅ successfully on 8a7d35c

Works good here.


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

@zero-24 zero-24 added this to the Joomla! 3.5.0 milestone Nov 7, 2015
@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Nov 7, 2015

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Nov 7, 2015
@infograf768
Copy link
Copy Markdown
Member

Can you do the same for Hathor?

roland-d added a commit that referenced this pull request Nov 10, 2015
Fixed long title in Smart Search: Indexed Content
@roland-d roland-d merged commit 8c05bb8 into joomla:staging Nov 10, 2015
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Nov 10, 2015
@pe7er pe7er deleted the com_finder-long-title-wordwrap 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