Opened 4 years ago
Last modified 2 years ago
#5383 new task (blessed)
Trac: Merge "Text Changes" component with the "ui-copy" focus
Reported by: | SergeyBiryukov | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Component: | Trac | Keywords: | |
Cc: |
Description (last modified by )
The ui-copy
focus was added to Trac in [9076]. Apparently the name is not clear enough for non-native English speakers and causes some confusion: #WP51080 is just a recent example, but there were some earlier examples too.
There is also a "Text Changes" component that is no longer actively used and can be retired in favor of the focus, per comment:2:ticket:4850, as text changes can belong to a particular component or span across multiple components.
Based on the above, I think the following would make sense:
- Rename the
ui-copy
focus totext-changes
to clarify its purpose. - Remove the "Text Changes" component in favor of the focus.
Change History (4)
#4
@
2 years ago
Rename the ui-copy focus to text-changes to clarify its purpose.
Unfortunately focuses are actually a text-field, not a category, and so renaming is not exactly possible.
We can delete the ui-copy
focus, and add a new text-changes
focus, but that won't migrate the existing tickets. Additionally, it's not possibly to bulk-edit tickets from the Trac UI to change the focus field, without a systems request to run a find-replace on the field.
Instead of a systems request, We might be able to add a method to our custom Trac WP-API, which will allow running a SQL against the Trac DB to do that find-replace.
Remove the "Text Changes" component in favor of the focus.
I'm not sure how Trac handles deleted components for existing tickets that are already closed on the component, I'd want to verify what happens there first before actioning this. I suspect it's fine and it'll continue to show the old component, just make it harder to query for them retroactively.
Thanks @SergeyBiryukov I concur, even as an english speaker it took me a while to realize
ui-copy
wasn't a focus on copy-paste actions in the UI.