? Pending

User tests: Successful: Unsuccessful:

avatar cpfeifer
cpfeifer
5 Feb 2017

Pull Request for Issue #13862

Summary of Changes

Minor CSS Adjustments which define input widths in the right sidebar of editing screens to provide a more consistent appearance.
Specifically:
Tag Select Input, Version Text Input and Version Note Text Input
Found in editor interface for various components.

Publish & Unpublish Date Select Inputs
Found in Module editor interface.

Testing Instructions

Enter item editing screen in the following components:
Articles, Categories, Menu Items, Banners, Contacts, News Feeds, Tags, Modules

This PR adjusts Note, Version Note, Tags, and Publish / Unpublish inputs. If there are other inputs with issues please let me know and I will make further adjustments.

Expected result

All parameter options found on the right side should be the same width in all components.

editor-inputs

Documentation Changes Required

None

avatar cpfeifer cpfeifer - open - 5 Feb 2017
avatar cpfeifer cpfeifer - change - 5 Feb 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 Feb 2017
Category Administration Templates (admin)
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 5 Feb 2017

I have tested this item successfully on 50f0276


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 5 Feb 2017 - Not tested
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 5 Feb 2017 - Tested successfully
avatar C-Lodder
C-Lodder - comment - 6 Feb 2017

If the note input is going to be made that small, it should be changed to a textarea. A note could potentially be a small paragraph, so using a small input seems pointless.

avatar cpfeifer
cpfeifer - comment - 6 Feb 2017

I don't disagree, but the issue in question was that the text inputs were hanging off to the right side. This is a minor detail in my opinion and I could go either way on it.

avatar C-Lodder
C-Lodder - comment - 6 Feb 2017

So rather than changing the width just to make it blend in, why don't we do the right thing and use the correct element? Will need a simple tweak in the XML file.

avatar cpfeifer
cpfeifer - comment - 6 Feb 2017

I can do that. I didn't mess with the input types at all but that makes a lot sense.

avatar tuts-22
tuts-22 - comment - 7 Feb 2017

I have tested the request and it works fine

avatar cpfeifer
cpfeifer - comment - 7 Feb 2017

I'll update this PR this week with the field types, it's a busy week for me.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 22 Feb 2017

@cpfeifer any News about this PR?

avatar brianteeman
brianteeman - comment - 20 May 2017

@cpfeifer it has been several months since you said you would be updating this PR. It now has multiple conflicts which also need resolving. If this isnt updated it will be closed in a few weeks time as there is no point in keeping an abandoned and non mergeable PR open. Thanks

avatar cpfeifer cpfeifer - change - 25 May 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-05-25 17:01:28
Closed_By cpfeifer
Labels Added: ?
avatar cpfeifer cpfeifer - close - 25 May 2017

Add a Comment

Login with GitHub to post a comment