?
avatar brianteeman
brianteeman
30 Jan 2016

It is not possible to select or create a new admin module on a mobile as the select filter to switch from site to admin modules is inside a div with class hidden-phone

avatar brianteeman brianteeman - open - 30 Jan 2016
avatar brianteeman
brianteeman - comment - 1 Feb 2016

I had hoped that it would be as simple as removing the hidden-mobile hidden-tablet from here https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_modules/layouts/joomla/searchtools/default/bar.php#L24

That works but doesnt look good


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 12 Feb 2016

@brianteeman i'm working on it (among other things). Tomorrow i should have it done.
See staging...andrepereiradasilva:com_modules-modules-searchtools

But i have two doubts:

  1. Is the modal view for modules still used? I didn't found any call to modules modal view in current Joomla staging.
  2. Is the ordering (drag and drop) of the modules in current staging working properly? When draging it disabled a lot of options, but are the right ones disabled?
avatar brianteeman
brianteeman - comment - 18 Feb 2016

Sorry I missed this reply
1. i think it may be used only in the loadmodule editor button
2. not sure what you mean


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 18 Feb 2016
  1. ok thanks. I think there is no way yet to put searchtools JLayouts on modals.
  2. is resolved now see #9120
avatar brianteeman
brianteeman - comment - 18 Feb 2016

The only thing I was referring to in this issue was that in mobile views the option to switch between site and admin is missing
mwpo 1

in #9101 for langauges you resolved it
w7cf

avatar andrepereiradasilva
andrepereiradasilva - comment - 18 Feb 2016

that also happens in menu items view.
When i have time i will try to do a PR for resolving that issue in all views.

avatar brianteeman
brianteeman - comment - 18 Feb 2016

Thanks - I will leave this open for now

avatar andrepereiradasilva
andrepereiradasilva - comment - 18 Feb 2016

PR #9151
Please test.

avatar brianteeman brianteeman - change - 18 Feb 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-02-18 16:56:46
Closed_By brianteeman
avatar brianteeman brianteeman - close - 18 Feb 2016

Add a Comment

Login with GitHub to post a comment