Skip to content

[5.2] Wrap long text to prevent layout displaying beyond viewport#44441

Merged
Hackwar merged 2 commits intojoomla:5.2-devfrom
Quy:indexer-debug
Nov 12, 2024
Merged

[5.2] Wrap long text to prevent layout displaying beyond viewport#44441
Hackwar merged 2 commits intojoomla:5.2-devfrom
Quy:indexer-debug

Conversation

@Quy
Copy link
Contributor

@Quy Quy commented Nov 11, 2024

Summary of Changes

Fix long text from displaying beyond viewport.

Testing Instructions

Install blog sample data.
Enable Debug System.
Go Components > Smart Search > Index.
Click Index > Index Debugging.
Select Smart Search - Content.
Click Index button.
See Output section.

Actual result BEFORE applying this Pull Request

Horizontal scrollbar.

Expected result AFTER applying this Pull Request

No horizontal scrollbar.

@ghost
Copy link

ghost commented Nov 12, 2024

I have tested this item ✅ successfully on 247b663


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

1 similar comment
@chmst
Copy link
Contributor

chmst commented Nov 12, 2024

I have tested this item ✅ successfully on 247b663


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

@chmst
Copy link
Contributor

chmst commented Nov 12, 2024

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Nov 12, 2024
@Hackwar Hackwar enabled auto-merge (squash) November 12, 2024 23:24
@Hackwar Hackwar added this to the Joomla! 5.2.2 milestone Nov 12, 2024
@Hackwar
Copy link
Member

Hackwar commented Nov 12, 2024

Thanks for this contribution. 😄

@Hackwar Hackwar merged commit b864b19 into joomla:5.2-dev Nov 12, 2024
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Nov 12, 2024
@Quy Quy deleted the indexer-debug branch November 12, 2024 23:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants