? Pending

User tests: Successful: Unsuccessful:

avatar rjharishabh
rjharishabh
20 May 2021

Pull Request for Issue # .

Summary of Changes

Remove <hr> in the frontend edit module

Testing Instructions

Try to edit module in frontend
edit

Actual result BEFORE applying this Pull Request

hr-before

Expected result AFTER applying this Pull Request

hr-after

Documentation Changes Required

No

avatar rjharishabh rjharishabh - open - 20 May 2021
avatar rjharishabh rjharishabh - change - 20 May 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 May 2021
Category Front End com_config
avatar brianteeman
brianteeman - comment - 20 May 2021

The line is there intentionally as a separator between different parts of the form

avatar rjharishabh
rjharishabh - comment - 20 May 2021

The line is there intentionally as a separator between different parts of the form

Is it necessary to separate this way?
I don't think so

avatar chmst
chmst - comment - 22 May 2021

@rjharishabh I agree that the hr looks more like a bug than a feature ;)

Have you an idea for grouping the input fields?

avatar brianteeman
brianteeman - comment - 22 May 2021

it will of course depend on the template as well

avatar rjharishabh
rjharishabh - comment - 23 May 2021

Sorry for the late reply
Is it really necessary to group
there is no grouping required between position and status (should be similar to backend)
And the note is a text field and options is an accordion (IMO it's already separated)

I don't know much about design

avatar sandramay0905 sandramay0905 - test_item - 24 May 2021 - Tested successfully
avatar sandramay0905
sandramay0905 - comment - 24 May 2021

I have tested this item successfully on 7936469


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

avatar ChristineWk ChristineWk - test_item - 29 May 2021 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 29 May 2021

I have tested this item successfully on 7936469


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

avatar rdeutz
rdeutz - comment - 30 May 2021

the hr looks odd but removing it is not the right way because we loosing the separation in the backend

avatar rdeutz rdeutz - close - 30 May 2021
avatar rdeutz rdeutz - change - 30 May 2021
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2021-05-30 09:10:20
Closed_By rdeutz
Labels Added: ?
avatar rjharishabh
rjharishabh - comment - 30 May 2021

@rdeutz this is for frontend.

avatar chmst
chmst - comment - 30 May 2021

@rjharishabht this whole layout needs some work. Thank you for opening this PR anyway, this was really useful, as obviously the layout was not properly analysed before.

avatar rjharishabh
rjharishabh - comment - 30 May 2021

@rjharishabht this whole layout needs some work. Thank you for opening this PR anyway, this was really useful, as obviously the layout was not properly analysed before.

Got it
Thanks

Add a Comment

Login with GitHub to post a comment