bug #9688
closedMake disabled fields copyable
Added by Katja Luther almost 3 years ago. Updated almost 3 years ago.
100%
Description
from #9287:
There is only one final issue which is valid for all disabled fields. Is it possible to generally make disabled fields copyable? Currently they only show the text but it is not possble to copy it. At some places this is already implemented. We probably should move this issue to a new ticket.
Related issues
Updated by Katja Luther almost 3 years ago
- Subject changed from make disabled fields copyable to Make disabled fields copyable
Updated by Katja Luther almost 3 years ago
- Status changed from New to Resolved
- Assignee changed from Katja Luther to Andreas Müller
- Target version changed from Unassigned CDM tickets to Release 5.25
please review.
Updated by Andreas Müller almost 3 years ago
- Copied from feature request #9287: Show term details in details view of term tree editor added
Updated by Andreas Müller almost 3 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
- % Done changed from 0 to 30
Copy from #9287:
AM:
For toggleable they still set the dirty flag (tested on Person details view with bulk editor). This is unwanted.
I do not yet fully understand why it is not possible to use the same technique as for ordinary text fields in toggleable fields.
KL:
The same technique is used for text fields and toggleable text fields but maybe I need to have a look on the event handling of the toggleable text field.
Updated by Katja Luther almost 3 years ago
Andreas Müller wrote:
Copy from #9287:
AM:
For toggleable they still set the dirty flag (tested on Person details view with bulk editor). This is unwanted.
I do not yet fully understand why it is not possible to use the same technique as for ordinary text fields in toggleable fields.
KL:
The same technique is used for text fields and toggleable text fields but maybe I need to have a look on the event handling of the toggleable text field.
The toggleable text field has additional modifyListener and keyListener, maybe this is the cause of the different behaviour.
Updated by Katja Luther almost 3 years ago
- Status changed from Feedback to Resolved
- % Done changed from 30 to 50
Applied in changeset taxeditor|5953dccec12018da410a14fca6d0d37c30822dc8.
Updated by Katja Luther almost 3 years ago
- Assignee changed from Katja Luther to Andreas Müller
- % Done changed from 50 to 30
This is fixed, please review.
Updated by Andreas Müller almost 3 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
Now an unprotected toggleable field can be edited. I tested with unprotected collector title of a team. But same result for e.g. name cache.
Updated by Katja Luther almost 3 years ago
- Status changed from Feedback to Resolved
- % Done changed from 30 to 50
Applied in changeset taxeditor|20e64571d5a2eb538996e867b9894f04360e7f95.
Updated by Andreas Müller almost 3 years ago
- Assignee changed from Katja Luther to Andreas Müller
I guess this is for review.
Updated by Andreas Müller almost 3 years ago
- Status changed from Resolved to Closed
- Assignee changed from Andreas Müller to Katja Luther
- % Done changed from 50 to 100
works as expected now.
Updated by Andreas Müller almost 3 years ago
- Has duplicate bug #8277: Username (and other fields) should allow c&p added
Updated by Andreas Müller almost 3 years ago
- Status changed from Closed to Feedback
- % Done changed from 100 to 80
I have to reopen this as now the Username and Group.name fields are editable again, but shouldn't. See also #8277.
Changing the username must not be possible. What is the reason why these fields are now editable again? Is it possible that there are other fields influenced in the same way that are now editable but they shouldn't? If yes, this is critical and needs to be fixed as soon as possible.
Updated by Katja Luther almost 3 years ago
Fixed it for the Userdetails but the group name never was disabled. I can do this as well, but maybe this is a little bit more difficult because it should be editable as long as it is not used anywhere?
Updated by Andreas Müller almost 3 years ago
- Status changed from Feedback to Resolved
- Assignee changed from Katja Luther to Andreas Müller
- % Done changed from 80 to 90
Katja Luther wrote:
Fixed it for the Userdetails but the group name never was disabled. I can do this as well, but maybe this is a little bit more difficult because it should be editable as long as it is not used anywhere?
Yes, for groupnames it is not so obvious. But both fields are marked as unique so the easiest way to guarantee this is to make the field not editable. On the other hand a groupname should have a semantic meaning so sometimes an admin may want to change the name to a better one.
So, yes, probably we should leave it editable. It is not a serious issue anyway.
I will check the username implementation and close the ticket then.
Updated by Andreas Müller almost 3 years ago
- Status changed from Resolved to Closed
- Assignee changed from Andreas Müller to Katja Luther
- % Done changed from 90 to 100
For usernames it works as expected.