RTC bug PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar nordmograph
nordmograph
15 Oct 2025

Pull Request for Issue # .

Summary of Changes

The width was hardcoded to 45%, regardless of the number of columns. This change makes it dynamic, distributing the width according to the number of fields, while reserving 8% for the action column.

Testing Instructions

Create a subform 'test' field with a repeatable.table layout and at least 3 (text for eg.) fields in it.
(layout="joomla.form.field.subform.repeatable-table") + subform_test.xml

Actual result BEFORE applying this Pull Request

2 first fields have 45% as width and the 3rd and following ones are compressed

Expected result AFTER applying this Pull Request

the width of ech field is a divisiion of the total width per the amount of fields

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar nordmograph nordmograph - open - 15 Oct 2025
avatar nordmograph nordmograph - change - 15 Oct 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Oct 2025
Category Layout
avatar nordmograph nordmograph - change - 15 Oct 2025
Labels Added: PR-5.4-dev
avatar richard67 richard67 - change - 15 Oct 2025
Title
Update repeatable-table.php
[5.4] Update repeatable-table.php
avatar richard67 richard67 - edited - 15 Oct 2025
avatar chmst
chmst - comment - 17 Oct 2025

Tested and works perfectly.
I’d round it to two decimal places - this looks a bit weird:
grafik

avatar nordmograph nordmograph - change - 23 Oct 2025
Labels Added: bug
avatar muhme
muhme - comment - 24 Oct 2025

@chmst Could you do an issue tracker test?

avatar chmst chmst - test_item - 24 Oct 2025 - Tested successfully
avatar chmst
chmst - comment - 24 Oct 2025

I have tested this item āœ… successfully on 3d4662a


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

avatar chmst
chmst - comment - 24 Oct 2025

Tested successfully: All columns have the same width.

grafik
avatar muhme
muhme - comment - 24 Oct 2025

@nordmograph I have tested this PR and see:

  • before PR: <th scope="col" style="width:45%">
  • with PR: <th scope="col" style="width:13.142857142857%">

However, there is no difference in the presentation for me on quick view in any of the three browsers (Firefox, Chrome, and Safari). All fields are present, and I can scroll horizontally before or after the PR. Could you please explain/show what bug this PR fixes?

shoot
avatar richard67
richard67 - comment - 24 Oct 2025

@muhme Maybe you can see the effect only on small screens, e.g. mobile phone?

avatar chmst
chmst - comment - 24 Oct 2025

It is the calculation - before: every column has with: 45% which is not correct if there are more than 2 cloumns. Now, all columns have a correctly calculated width.
Users don't see a difference because the fields enforce the width they need and hoizontal scrolling is necessary.

avatar muhme
muhme - comment - 31 Oct 2025

@nordmograph and other testers: How to reproduce the problem? Which browser, under what circumstances, with a screenshot?

2 first fields have 45% as width and the 3rd and following ones are compressed

avatar Fedik
Fedik - comment - 31 Oct 2025

@muhme try use different field types as @chmst did.

The fix mostly cosmetic, the width is ignored by Table (table use it as "relative width"), and example for all "text input" it will be equal for each cell.
With different content in the cell the result should be more visible.

avatar muhme
muhme - comment - 31 Oct 2025

Even with different content, all columns are equal in size:
before
Is there is not enough horicontal space, there is horicontel scrolling needed, but nothing is compressed:
before-with-scrolling
After applying the PR with Patch Tester it looks identical as before. Sorry, I can't reproduce the problem.

avatar Fedik
Fedik - comment - 31 Oct 2025

In some reason I expected it will be relive to content width, but as all columns have same width 45% the table make it equal also for any amount of columns.

But, as I said it just cosmetic changes, even if we set width:100% the table will "make it fit".

avatar muhme muhme - test_item - 8 Nov 2025 - Tested successfully
avatar muhme
muhme - comment - 8 Nov 2025

I have tested this item āœ… successfully on 21a6aed

Tested with JBT, the width is calculated dynamically for the number of fields.


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

avatar muhme muhme - change - 8 Nov 2025
The description was changed
Status Pending Ready to Commit
avatar muhme
muhme - comment - 8 Nov 2025

RTC


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

avatar muhme muhme - edited - 8 Nov 2025
avatar richard67 richard67 - change - 8 Nov 2025
Labels Added: RTC
avatar richard67 richard67 - change - 8 Nov 2025
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2025-11-08 16:38:31
Closed_By richard67
avatar richard67 richard67 - close - 8 Nov 2025
avatar richard67 richard67 - merge - 8 Nov 2025
avatar richard67
richard67 - comment - 8 Nov 2025

Thanks @nordmograph for this fix, @chmst and @muhme for testing and @Fedik for support.

Add a Comment

Login with GitHub to post a comment