Feeling Lucky
?
?
[#30519] - Fix page break title when title attribute is after class
- Fixed in Code Base
- 31 Aug 2020
- Medium
- Build: staging
- # 30519
- Diff
- SharkyKZ:j3/bug/pagebreak-title
User tests: Successful: Unsuccessful:
Fixes #30518.
Summary of Changes
Fixes page break title when title attribute is placed after class attribute containing system-pagebreak class.
Testing Instructions
Use CodeMirror as editor.
Create a multipage article.
Change attribute order in page break's <hr> tag so title is after class, e.g.:
<p>Page 1.</p>
<hr alt="Page 2 Alt" class="system-pagebreak" title="Page 2 Title" />
<p>Page 2.</p>
<hr title="Page 3 Title" alt="Page 3 Alt" class="system-pagebreak" />
<p>Page 3.</p>
View Page 2 and Page 3 of the article.
Actual result BEFORE applying this Pull Request
Title from title attribute shown only on Page 3.
Expected result AFTER applying this Pull Request
Titles from title attribute are shown on both pages.
Documentation Changes Required
No.
Votes
# of Users Experiencing Issue
1/1
Average Importance Score
5.00
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
viocassel
- comment
- 31 Aug 2020
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30519.
| Status | Pending | ⇒ | Ready to Commit |
alikon
- comment
- 31 Aug 2020
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30519.
zero-24
- comment
- 31 Aug 2020
Merging thanks
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-08-31 15:52:36 |
| Closed_By | ⇒ | zero-24 | |
| Labels |
Added:
?
?
|
||
I have tested this item✅ successfully on cac2ab1
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30519.