? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
17 Jan 2020

Simple PR adds clarity and improves readability

dont forget npm i

example on smart search

before

image

after

image

avatar brianteeman brianteeman - open - 17 Jan 2020
avatar brianteeman brianteeman - change - 17 Jan 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 Jan 2020
Category Administration Templates (admin)
avatar ciar4n ciar4n - test_item - 17 Jan 2020 - Tested successfully
avatar ciar4n
ciar4n - comment - 17 Jan 2020

I have tested this item successfully on f792bff


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27554.

avatar jwaisner jwaisner - test_item - 17 Jan 2020 - Tested successfully
avatar jwaisner
jwaisner - comment - 17 Jan 2020

I have tested this item successfully on f792bff


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27554.

avatar SharkyKZ SharkyKZ - change - 17 Jan 2020
Status Pending Ready to Commit
Labels Added: ?
avatar SharkyKZ
SharkyKZ - comment - 17 Jan 2020

RTC.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27554.

avatar Quy
Quy - comment - 17 Jan 2020

For RTL, it should be right-aligned.

avatar infograf768 infograf768 - change - 17 Jan 2020
Status Ready to Commit Pending
avatar infograf768
infograf768 - comment - 17 Jan 2020

Back to pending until rtl is taken care of to avoid another pr for mini stuff


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27554.

avatar infograf768
infograf768 - comment - 18 Jan 2020

Suggestion:

[...]
  z-index: $zindex-tooltip;

  [dir=ltr] & {
    text-align: left;
  }
  
  [dir=rtl] & {
    text-align: right;
  }
avatar brianteeman
brianteeman - comment - 18 Jan 2020

Thanks. I will update soon

6a3553f 18 Jan 2020 avatar brianteeman rtl
avatar brianteeman
brianteeman - comment - 18 Jan 2020

updated for rtl

avatar infograf768 infograf768 - change - 18 Jan 2020
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 18 Jan 2020

As change is obvious, setting rtc


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27554.

avatar brianteeman brianteeman - change - 18 Jan 2020
Labels Added: ?
avatar brianteeman
brianteeman - comment - 18 Jan 2020

Done it - its hard to check this sort of thing from my phone

avatar wilsonge wilsonge - change - 18 Jan 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-01-18 14:28:18
Closed_By wilsonge
avatar wilsonge wilsonge - close - 18 Jan 2020
avatar wilsonge wilsonge - merge - 18 Jan 2020
avatar wilsonge
wilsonge - comment - 18 Jan 2020

Thanks!

avatar brianteeman
brianteeman - comment - 18 Jan 2020

Thanks

Add a Comment

Login with GitHub to post a comment