Feeling Lucky
?
NPM Resource Changed
?
[#37877] - [4.1] tag active focus
- Fixed in Code Base
- 28 May 2022
- Medium
- Build: 4.1-dev
- # 37877
- Diff
- brianteeman:tags
User tests: Successful: Unsuccessful:
Pull Request for Issue #37864 .
Summary of Changes
add a margin around the tags so that the focus ring is displayed correctly
Testing Instructions
This is a css change so either use a pre-built package or npm run build:css
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
Documentation Changes Required
n/a
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Repository NPM Change |
Kostelano
- comment
- 24 May 2022
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37877.
| Status | Pending | ⇒ | Ready to Commit |
alikon
- comment
- 24 May 2022
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37877.
| Labels |
Added:
?
?
|
||
richard67
- comment
- 26 May 2022
@brianteeman After I had updated your branch to make the npm step pass in drone, the scss code style check fails in drone: https://ci.joomla.org/joomla/joomla-cms/54621/1/24 .
brianteeman
- comment
- 26 May 2022
Thanks for fixing drone. I thought it might have been better as short hand but couldnt remember if we just left that to the compiler..
| Labels |
Added:
NPM Resource Changed
|
||
sandewt
- comment
- 26 May 2022
Thanks
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-05-28 20:28:16 |
| Closed_By | ⇒ | Quy |
Quy
- comment
- 28 May 2022
Thanks
I have tested this item✅ successfully on 3fa7b26
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37877.