? Language Change PR-4.3-dev Pending

User tests: Successful: Unsuccessful:

avatar HLeithner
HLeithner
29 Jun 2022

Summary of Changes

Add a column to the subform custom field which allows to select they layout for editing

Testing Instructions

  • create a subform
  • select the type of the edit layout
  • layout is used on editing

Actual result BEFORE applying this Pull Request

Not possible

Expected result AFTER applying this Pull Request

Works

Documentation Changes Required

possible

avatar HLeithner HLeithner - open - 29 Jun 2022
avatar HLeithner HLeithner - change - 29 Jun 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 29 Jun 2022
Category Administration Language & Strings Front End Plugins
avatar brianteeman
brianteeman - comment - 29 Jun 2022

This is a good example of issues with the editorconfig that we now have.

The file is using tabs but your new code is using spaces :(

avatar HLeithner
HLeithner - comment - 30 Jun 2022

This is a good example of issues with the editorconfig that we now have.

The file is using tabs but your new code is using spaces :(

hmm you are right, seems I was to optimistic for the editorconfig.

I will write a PR that only php files uses spaces.

83ff61c 30 Jun 2022 avatar HLeithner cs
avatar HLeithner HLeithner - change - 30 Jun 2022
Labels Added: Language Change ?
avatar HLeithner
HLeithner - comment - 30 Jun 2022

Now I used tabs in the xml

avatar crystalenka
crystalenka - comment - 28 Oct 2022

Works well, and is a need! Just would recommend moving the field to Options tab in the Form Options field group, as that pattern is already established in other field types like radio:

avatar crystalenka
crystalenka - comment - 28 Oct 2022

Hahaha my change made something not work. Sorry. ? I'll open another PR against your branch after Amelia is in bed and I can fix it.

avatar Hackwar
Hackwar - comment - 28 Oct 2022

Since 4.2 only accepts Bugfixes and this is a new feature (or rather an improvement), I've rebased the PR to 4.3-dev and updated it.

avatar Hackwar Hackwar - change - 28 Oct 2022
Labels Added: PR-4.3-dev
avatar Quy Quy - change - 27 Jan 2023
Labels Removed: ?
avatar Quy Quy - test_item - 27 Jan 2023 - Tested successfully
avatar Quy
Quy - comment - 27 Jan 2023

I have tested this item successfully on f25d759


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

avatar carlitorweb carlitorweb - test_item - 28 Jan 2023 - Tested successfully
avatar carlitorweb
carlitorweb - comment - 28 Jan 2023

I have tested this item successfully on f25d759


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

avatar carlitorweb
carlitorweb - comment - 28 Jan 2023
avatar Quy Quy - change - 28 Jan 2023
Status Pending Ready to Commit
avatar Quy
Quy - comment - 28 Jan 2023

RTC


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

avatar Quy Quy - change - 28 Jan 2023
Labels Added: ?
avatar HLeithner HLeithner - change - 28 Jan 2023
The description was changed
avatar HLeithner HLeithner - edited - 28 Jan 2023
avatar HLeithner
HLeithner - comment - 28 Jan 2023

Thanks @carlitorweb for the documentation update

avatar rdeutz rdeutz - close - 28 Jan 2023
avatar rdeutz rdeutz - merge - 28 Jan 2023
avatar rdeutz rdeutz - change - 28 Jan 2023
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-01-28 12:38:17
Closed_By rdeutz

Add a Comment

Login with GitHub to post a comment