Ticket #3159 (assigned enhancement)

Opened 5 years ago

Last modified 7 months ago

Skin files: fix badly chosen keywords

Reported by: egmont Owned by: andrew_b
Priority: minor Milestone: 4.8.22
Component: mc-skin Version: 4.8.11
Keywords: Cc:
Blocked By: Blocking:
Branch state: no branch Votes for changeset:

Description

Inspired by #2920 I started adding comments to default.ini. But it would be better if the names were more self-explanatory so we needed fewer comments.

I propose to change at least these names:

gauge -> progressbar (seems people have no clue what gauge could stand for)
reverse -> currentdir (keyword should describe the role, not the look)
[core] disabled -> [dialog] disabled (or "ddisabled" with double d for consistency?), it's only used in dialogs
helpslink -> helpselectedlink

Change History

comment:1 Changed 2 years ago by zaytsev

  • Status changed from new to assigned
  • Owner set to andrew_b
  • Milestone changed from Future Releases to 4.8.19

comment:2 Changed 23 months ago by egmont

See #3759 for a few more.

comment:3 Changed 23 months ago by egmont

Plus:

[editor] -> editbold: just like reverse, the name should describe the role (search result) rather than look.

[editor] -> bookmarkfound: refers to technical implementation rather than user visible role. It's for highlighting search results at "find all". Nowhere does the UI say it has anything to do with bookmarks. It's irrelevant that the source code piggybacks on the same engine.

Some names duplicate the section name (e.g. [editor] -> editstuff, [menu] -> menuwhatever), some others don't (e.g. [diffviewer] -> added). The latter makes more sense to me; if the names are prefixed with the section name then why do we have sections?

comment:4 Changed 13 months ago by zaytsev

  • Milestone changed from 4.8.20 to 4.8.21

comment:5 Changed 7 months ago by zaytsev

  • Milestone changed from 4.8.21 to 4.8.22
Note: See TracTickets for help on using tickets.