? Success

User tests: Successful: Unsuccessful:

avatar Bakual
Bakual
3 Nov 2016

Currently, when editing a field the description (tooltip) of the field is on the first tab and the label is on the third one. Imho they should be together and since the label is a required field it should be on the first tab.

Summary of Changes

  • Moving the label field to first tab right below the type field. This way the required fields are the first ones.
  • Moving the description right below the label one so they are together.
  • Changed the type of the default value field from textarea to text. I can't think of a usecase where the default value would span multiple rows and thus text should be sufficient always. Sidenote here: Does the default_value field really need a raw filter? That could potentially open a security issue if the input isn't filtered in another place.

This has the good sideeffect that the fieldparameters (those depending on the selected field type) aren't loaded between the common parameters default_value and description, they now appear at below all other parameters.

Testing Instructions

Edit a custom field and check the moved parameters and that they still work as expected.

Documentation Changes Required

None

avatar Bakual Bakual - open - 3 Nov 2016
avatar Bakual Bakual - change - 3 Nov 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 3 Nov 2016
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 3 Nov 2016
Category Administration Components
avatar jeckodevelopment jeckodevelopment - change - 3 Nov 2016
Category Administration Components Administration com_fields Components
avatar Bakual
Bakual - comment - 3 Nov 2016

Some pictures so it's more clear ?

Before:
before
before2

After:
after

avatar brianteeman
brianteeman - comment - 3 Nov 2016

agree 100%

a required field should never be buried on a tab

On 3 November 2016 at 19:51, Thomas Hunziker notifications@github.com
wrote:

Some pictures so it's more clear ?

Before:
[image: before]
https://cloud.githubusercontent.com/assets/1018684/19982656/31eca2c6-a207-11e6-9cf5-e077645310dc.PNG
[image: before2]
https://cloud.githubusercontent.com/assets/1018684/19982661/39469842-a207-11e6-9cc5-6a7fc9ec4b0a.PNG

After:
[image: after]
https://cloud.githubusercontent.com/assets/1018684/19982664/3e101344-a207-11e6-9420-147124423a31.PNG


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#12740 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8TXTu1S2CbKV7WR6E7EMBX_djvXeks5q6js5gaJpZM4Ko1i1
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
https://brian.teeman.net/ http://brian.teeman.net/

avatar alikon
alikon - comment - 3 Nov 2016

I have tested this item successfully on 77457a4


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

avatar alikon alikon - test_item - 3 Nov 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 3 Nov 2016

Oh wait a moment. I remember discussing this one before. See joomla-projects/custom-fields#22
By default the label is autocompleted with the value of the field name.

It originally was on the first tab and it was very confusing.

Just watch the label get autocompleted as you have typed the field name

TBH I didnt think the field should even exist at all. The use case presented joomla-projects/custom-fields#22 (comment) suggested that the only time you might change it was for customisation of the visual presentation in which case it made sense to move it to the tab responsible for visual presentation.

avatar andrepereiradasilva andrepereiradasilva - test_item - 3 Nov 2016 - Tested successfully
avatar andrepereiradasilva
andrepereiradasilva - comment - 3 Nov 2016

I have tested this item successfully on 77457a4


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

avatar ggppdk
ggppdk - comment - 3 Nov 2016

Maybe change the description of 'label':

Label of field is used when editing field in forms and when viewing field values. This is different than field title, that is only used in backend fields management

avatar Bakual
Bakual - comment - 3 Nov 2016

It is confusing for sure.
I think the label and description have to stay together as the description is the tooltip of the label.
I know the label is autocompleted, which is a bit strange by itself. Personally I would make the label optional and use the title if the label is not specified. Then it could go to the option tab.
But as long as it has the required flag, I don't want to have it on third tab and the description on the first one.

avatar brianteeman
brianteeman - comment - 3 Nov 2016

I still think it should not be required and it should be on the third tab
and the code should use the field name by default

On 3 November 2016 at 22:57, Thomas Hunziker notifications@github.com
wrote:

It is confusing for sure.
I think the label and description have to stay together as the description
is the tooltip of the label.
I know the label is autocompleted, which is a bit strange by itself.
Personally I would make the label optional and use the title if the label
is not specified. Then it could go to the option tab.
But as long as it has the required flag, I don't want to have it on third
tab and the description on the first one.


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#12740 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8eQzVaDe_stlR4wz5s0fThC68DWZks5q6mbCgaJpZM4Ko1i1
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
https://brian.teeman.net/ http://brian.teeman.net/

avatar infograf768
infograf768 - comment - 4 Nov 2016

The label is a very important aspect as it may become a constant for a language string.
As well for the description. They should be together, as we do in the xmls.
As we do not accept any type of text as labels (parse ini), taking it off would be a huge mistake.

avatar jeckodevelopment
jeckodevelopment - comment - 4 Nov 2016

So what to do with this PR?

avatar infograf768
infograf768 - comment - 4 Nov 2016

For me, should be RTC

avatar jeckodevelopment jeckodevelopment - change - 4 Nov 2016
Status Pending Ready to Commit
avatar jeckodevelopment
jeckodevelopment - comment - 4 Nov 2016

RTC


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

avatar jeckodevelopment jeckodevelopment - change - 4 Nov 2016
Milestone Added:
avatar jeckodevelopment jeckodevelopment - change - 4 Nov 2016
Milestone Added:
avatar jeckodevelopment
jeckodevelopment - comment - 4 Nov 2016

RTC


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

avatar rdeutz rdeutz - close - 5 Nov 2016
avatar rdeutz rdeutz - merge - 5 Nov 2016
avatar rdeutz rdeutz - reference | 3d3b767 - 5 Nov 16
avatar rdeutz rdeutz - merge - 5 Nov 2016
avatar rdeutz rdeutz - close - 5 Nov 2016
avatar rdeutz rdeutz - change - 5 Nov 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-11-05 09:27:56
Closed_By rdeutz
avatar Bakual Bakual - head_ref_deleted - 5 Nov 2016
avatar PhilETaylor PhilETaylor - reference | 7ea3a7b - 6 Nov 16
avatar laoneo
laoneo - comment - 7 Nov 2016

Why did you change the default_value parameter to a text field? What's the benefit?

avatar Bakual
Bakual - comment - 7 Nov 2016

From the PR description:

Changed the type of the default value field from textarea to text. I can't think of a usecase where the default value would span multiple rows and thus text should be sufficient always.

avatar ggppdk
ggppdk - comment - 7 Nov 2016

I can't think of a usecase where the default value would span multiple rows and thus text should be sufficient always.

Not always

In most cases "text" is enough to contain the default value,
but some fields may need a larger default value, e.g. an editor field or 3rd party fields,
for these cases "text" will make it a little difficult to edit

  • it will depend on user's / site's needs

a default value of type "textarea" with 2 rows that user can drag to enlarge would be best

avatar Bakual
Bakual - comment - 7 Nov 2016

I think having a two line default value is looking bad on the form and gives wrong clues to the user. But if you see a use case and think it's worth having a textarea to enter a 1, then go ahead and change it back :)

avatar ggppdk
ggppdk - comment - 7 Nov 2016

I think having a two line default value is looking bad on the form and gives wrong clues to the user ...

You are right about this, great majority of fields will need 1 line text box as default value
that is why it would have been best if default value parameter was in the XML file of the field type,
instead of this being a common parameter, and indeed the great majority of fields should need only "text"

also other reason for it to be in the XML file is:

  • custom parameter description about default value
  • and for some fields it can "default values" and not "default value"
avatar laoneo
laoneo - comment - 7 Nov 2016

I can't remember, but there was a reason I put is as text area, PR which reverts it back, #12811.

Add a Comment

Login with GitHub to post a comment