Feeling Lucky
?
?
[#32415] - Add styling to article index (toc)
- Fixed in Code Base
- 15 Feb 2021
- Medium
- Build: 4.0-dev
- # 32415
- Diff
- drmenzelit:4.0-dev-article-index
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
Added BS5 classes for margin and padding to article index (toc)
Added color to active link
Run npm (scss changes)
Testing Instructions
Create an article with page breaks and article index (table of content)
Actual result BEFORE applying this Pull Request
The article index has no margin to the article text, the links have no adding and the active link is not discernible:
Expected result AFTER applying this Pull Request
The article index has a margin (works in lrt too), the links have some padding and the active link is visually discernible:
Documentation Changes Required
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
| Labels |
Added:
J4 Issue
?
|
||
| Category | Front End Plugins | ⇒ | Front End Plugins Templates (site) NPM Change |
| Title |
|
||||||
brianteeman
- comment
- 14 Feb 2021
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32415.
chmst
- comment
- 14 Feb 2021
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32415.
| Status | Pending | ⇒ | Ready to Commit |
chmst
- comment
- 14 Feb 2021
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32415.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-02-15 19:36:15 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
Removed: J4 Issue |
||
wilsonge
- comment
- 15 Feb 2021
Thanks!
@drmenzelit We don't use labels "... Issue" on PR's, only on issues.