Feeling Lucky
NPM Resource Changed
?
?
[#27416] - [4.0] Move Searchtools styling to template
- Fixed in Code Base
- 6 Jan 2020
- Medium
- Build: 4.0-dev
- # 27416
- Diff
- Fedik:searchtools-move-to-tmpl
- Pending Hound Hound is busy sniffing around... Details
User tests: Successful: Unsuccessful:
Summary of Changes
This will move searchtools styling to the template files.
More info in #27384 comments.
Testing Instructions
Apply patch, run npm install
Expected result
All works
Actual result
Almost works, with some rtl issue
@infograf768 please check
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration Templates (admin) JavaScript Repository NPM Change Libraries |
infograf768
- comment
- 6 Jan 2020
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27416.
jwaisner
- comment
- 6 Jan 2020
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27416.
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
?
NPM Resource Changed
|
||
SharkyKZ
- comment
- 6 Jan 2020
RTC.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27416.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-01-06 22:30:21 |
| Closed_By | ⇒ | HLeithner | |
| Labels |
Added:
?
|
||
HLeithner
- comment
- 6 Jan 2020
Thanks
Fedik
- comment
- 7 Jan 2020
@infograf768 it looks good to me:
LTR:
RTL:
Let's go for it.
We can correct some more stuff after merge.
Some are in _toolbar.scss
@wilsonge @HLeithner