NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar ciar4n
ciar4n
14 May 2021

Pull Request for Issue # .

Summary of Changes

  • Remove multiple lines between fieldsets
  • Remove excess padding around fieldsets

Before

image

After

image

avatar ciar4n ciar4n - open - 14 May 2021
avatar ciar4n ciar4n - change - 14 May 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 May 2021
Category Administration Templates (admin) NPM Change
avatar brianteeman
brianteeman - comment - 14 May 2021

Personally I find it clearer before this PR to see which fields are in a group - especially on single column pages such as article options.

image

image

avatar richard67
richard67 - comment - 14 May 2021

Does this PR solve issue #33548 , or is that something else?

avatar ciar4n
ciar4n - comment - 15 May 2021

@richard67 That issues appears to be just for the frontend where this is for the backend.

@brianteeman I dont think wrapping everything in boxes is the answer if you want more distinction between fieldsets.

avatar brianteeman
brianteeman - comment - 15 May 2021

I'm open to any suggestion

avatar ceford
ceford - comment - 15 May 2021

I prefer the status quo - the legend in a complete outline of the items it encloses. Can we vote this pr down?


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

avatar ciar4n
ciar4n - comment - 17 May 2021

I'm open to any suggestion

Considering this PR is one, how many do you want?

avatar ciar4n
ciar4n - comment - 17 May 2021

Closing considering lack of interest.

avatar ciar4n ciar4n - change - 17 May 2021
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2021-05-17 21:39:54
Closed_By ciar4n
Labels Added: NPM Resource Changed ?
avatar ciar4n ciar4n - close - 17 May 2021

Add a Comment

Login with GitHub to post a comment