Feeling Lucky
NPM Resource Changed
?
?
[#27829] - [4.0] minicolors
- Fixed in Code Base
- 11 Apr 2020
- Low
- Build: 4.0-dev
- # 27829
- Diff
- brianteeman:minicolors
- Pending Hound Hound is busy sniffing around... Details
User tests: Successful: Unsuccessful:
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration Templates (admin) Repository NPM Change |
brianteeman
- comment
- 6 Feb 2020
thats the vendor supplied css not the override
C-Lodder
- comment
- 6 Feb 2020
ah my bad
jwaisner
- comment
- 6 Feb 2020
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27829.
| Priority | Medium | ⇒ | Low |
richard67
- comment
- 11 Apr 2020
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27829.
| Status | Pending | ⇒ | Ready to Commit |
richard67
- comment
- 11 Apr 2020
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27829.
| Labels |
Added:
?
?
NPM Resource Changed
|
||
dgrammatiko
- comment
- 11 Apr 2020
@richard67 RTC? Really??? Did you ready any of my comments?
richard67
- comment
- 11 Apr 2020
@dgrammatiko Yes I did read them but I'm not deep enough into these things to tell what is right and what is wrong. I only can see this PR solves an issue.
@wilsonge Should I revert the RTC until the discussion is clarified?
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-04-11 21:23:58 |
| Closed_By | ⇒ | wilsonge | |
| Labels | |||
wilsonge
- comment
- 11 Apr 2020
This will do. It can be followed up with a vendor specific override in a future PR
brianteeman
- comment
- 11 Apr 2020
thanks <200 now
The CSS file is already being loaded here: https://github.com/joomla/joomla-cms/blob/4.0-dev/layouts/joomla/form/field/color/advanced.php#L76