Skip to content

[4.0] Add client property to module positions field#24698

Merged
wilsonge merged 1 commit intojoomla:4.0-devfrom
SharkyKZ:j4/modulesPositionEdit
Apr 23, 2019
Merged

[4.0] Add client property to module positions field#24698
wilsonge merged 1 commit intojoomla:4.0-devfrom
SharkyKZ:j4/modulesPositionEdit

Conversation

@SharkyKZ
Copy link
Contributor

Pull Request for Issue #24575.

Summary of Changes

Alternative PR to #24583. Adds client property to module position edit field.

Testing Instructions

Edit a module from:

  1. Control Panel
  2. From module list in backend
  3. From frontend

Inspect positions in Position field.

Expected result

Positions should be shown based on whether site or admin module is being edited.

Actual result

When editing an admin module from Control Panel, site positions are shown.

Documentation Changes Required

IDK.

@infograf768
Copy link
Member

I have tested this item ✅ successfully on a8f0a5b


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

@ghost
Copy link

ghost commented Apr 23, 2019

@chmst can you please test?

@wilsonge wilsonge merged commit bcfb107 into joomla:4.0-dev Apr 23, 2019
@wilsonge
Copy link
Contributor

wilsonge commented Apr 23, 2019

I'm much happy with this. Merging. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants