Bug 438948 - Create/Edit Resource Bundle dialogue is using the name "Brushes" for brushtips type
Summary: Create/Edit Resource Bundle dialogue is using the name "Brushes" for brushtip...
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Resource Management (other bugs)
Version First Reported In: 4.4.5
Platform: unspecified All
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-06-20 13:54 UTC by Tyson Tan
Modified: 2021-06-22 15:13 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Create Resource Bundle dialogue brushtip type is named as Brushes (96.20 KB, image/png)
2021-06-20 13:54 UTC, Tyson Tan
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tyson Tan 2021-06-20 13:54:31 UTC
Created attachment 139540 [details]
Create Resource Bundle dialogue brushtip type is named as Brushes

Krita's Create/Edit Resource Bundle dialogue is using the name "Brushes" for brushtip type.

Since they are precisely brushtips, we should name them here as such. Many users are having trouble telling the differences between "Brushes" and "Brush Presets".

I think "Brush Tip Images" is more accurate instead of "Brushes/brushtips". It'll help users to understand what they really are, and thus not mixing them with the Brush Presets.
Comment 1 Tiar 2021-06-21 16:39:33 UTC
The current name for the brush tip images resource type name is "Brush Tips" and it should be used everywhere (here it was a mistake), and the current paintoppresets/brush presets resource type name is "Brush Presets". If that should be changed, then it needs wider discussion or at least Halla's approval.

However the "Brushes" in the bundle manager is easy to fix, and I guess *removing* strings is fine in the string freeze, right? (It will be replaced by strings defined in other parts of the code, namely ResourcesTypes.h, which is what all other parts of Krita use). If you confirm, I'll push a fix for that today.
Comment 2 Tyson Tan 2021-06-22 01:27:30 UTC
Hi Tiar, I agree to just fix the "Brushes" to Brush Tips".
Comment 3 Tiar 2021-06-22 15:07:50 UTC
Git commit cf588a7f0c41d9bf52322c39b269415b0dc004bd by Agata Cacko.
Committed on 22/06/2021 at 15:07.
Pushed by tymond into branch 'master'.

Fix duplicate resource type names in DlgCreateBundle

Before this commit, resource type names in DlgCreateBundle
were hardcoded, usually duplicates from the strings in
ResourceName namespace in ResourceTypes.h.
This commit replaces them with strings from ResourceTypes.h.

M  +7    -8    plugins/extensions/resourcemanager/dlg_create_bundle.cpp

https://invent.kde.org/graphics/krita/commit/cf588a7f0c41d9bf52322c39b269415b0dc004bd
Comment 4 Tyson Tan 2021-06-22 15:13:05 UTC
Thanks Tiar! :D