Ticket #3159 (assigned enhancement)

Opened 6 years ago

Last modified 5 days ago

Skin files: fix badly chosen keywords

Reported by: egmont Owned by: andrew_b
Priority: minor Milestone: 4.8.26
Component: mc-skin Version: 4.8.11
Keywords: Cc: ales.janda@…
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 4 years ago by zaytsev

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

comment:2 Changed 3 years ago by egmont

See #3759 for a few more.

comment:3 Changed 3 years 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 3 years ago by zaytsev

  • Milestone changed from 4.8.20 to 4.8.21

comment:5 Changed 2 years ago by zaytsev

  • Milestone changed from 4.8.21 to 4.8.22

comment:6 Changed 19 months ago by zaytsev

  • Milestone changed from 4.8.22 to 4.8.23

comment:7 Changed 13 months ago by zaytsev

  • Milestone changed from 4.8.23 to 4.8.24

comment:8 Changed 5 days ago by zaytsev

  • Milestone changed from 4.8.25 to 4.8.26

comment:9 Changed 5 days ago by kybl

  • Cc ales.janda@… added
Note: See TracTickets for help on using tickets.