SabakiHQ / Sabaki

An elegant Go board and SGF editor for a more civilized age.
https://sabaki.yichuanshen.de/
MIT License
2.41k stars 377 forks source link

Feature Suggestion(s): `Manage Games` Improvements #599

Open psygo opened 5 years ago

psygo commented 5 years ago

Some of my suggestions were discussed in Issue #424. However, that was kind of a confusing discussion, so I will try to start anew.

The improvements I would like to suggest for the Manage Games feature are — I don't know if they are actually possible due to limitations imposed by sgf standards:

If all or some of these features seem reasonable, perhaps it would be to later put them into separate issues. But it would be useful to have a single issue to serve as a hub for the bigger picture discussion.

yishn commented 5 years ago
  • A title for each game in the collection.

You can already give each game a name in the game info drawer. The name field value will be shown in the game collection manager.

  • As a subfeature, there could also be a toggle to see the games in the collection as a list, where the titles would differentiate each one.

This is #146

  • A toggle button where you can set the preview to display the beginning of the games or their end. [...] This spoiling can even be a nuissance for normal games, because sometimes you don't want to see the final position.

We don't show the end position per se. Just 30 moves into the game, so it wouldn't be a spoiler for normal games. But of course, for Tsumegos, that would be the end position. For normal games, displaying the beginning of the games would mostly just be a blank board which isn't helpful information at all. Maybe we can just check how long the game is instead of having an option.

psygo commented 5 years ago

You can already give each game a name in the game info drawer. The name field value will be shown in the game collection manager.

You are correct. I hadn't noticed it. Nevertheless, I think it would also be intuitive and comfortable from a user's standpoint to have a way of renaming the game directly on the Manage Games preview. For example, this could be achieved by adding a Rename Game on the Right Click submenu.

For normal games, displaying the beginning of the games would mostly just be a blank board which isn't helpful information at all.

That's one of the reasons why I suggested a toggle button or another preference item. If the user feels like the preview isn't adding any information because it's only showing the initial position, then he toggles it.

I think it would be very difficult for the developer to sort of somehow guess which position number should be shown. It would be far easier if the user could choose what type of preview he wants.

psygo commented 5 years ago

By the way, it is a bit off-topic, but I would like to suggest another name for the label enhancement. Wouldn't something like feature suggestion be more accurate and specific? enhancement could mean that I'm just trying to improve the code for example.

I would also suggest adding brief descriptions to the labels so this type of "confusion" could be solved without you having to address it repeatedly.

Terkwood commented 5 years ago

Hope you don't mind a third party jumping in on a secondary topic, but I like "enhancement" as a label, since it aligns with the default labels used by github projects. I think the meaning is clear. Cheers.

On Sun, Aug 25, 2019, 22:05 Philippe Fanaro notifications@github.com wrote:

By the way, it is a bit off-topic, but I would like to suggest another name for the label enhancement. Wouldn't something like feature suggestion be more accurate and specific? enhancement could mean that I'm just trying to improve the code for example.

I would also suggest adding brief descriptions to the labels so this type of "confusion" could be solved without you having to address it repeatedly.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/SabakiHQ/Sabaki/issues/599?email_source=notifications&email_token=AJIPHCA3GECRMEXRAPQUYNLQGM26PA5CNFSM4IPHO7P2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5DB52Y#issuecomment-524689131, or mute the thread https://github.com/notifications/unsubscribe-auth/AJIPHCFJY3LU4A2FTBWOR4LQGM26PANCNFSM4IPHO7PQ .

psygo commented 5 years ago

Trailing off a bit still, but...

I still disagree, @Terkwood... Some practices are just common but not good in my opinion. It is true that enhancement is default, but what does it mean for the labeler or the developer? Code improvement? Documentation improvement? Performance improvement? Improvement in what sense? I think it is very vague.

For instance, Collins Dictionary defines "enhancement" (as does the Cambridge Dictionary also) with the preposition "of", meaning it needs a object to complete its meaning:

The enhancement of something is the improvement of it in relation to its value, quality, or attractiveness.

Anyway, many of the major libaries I regularly use still have that label, but they do use a myriad of much more specific labels, e.g., feature-request or Performance. Some of these libraries are:

If it is necessary to keep the enhancement label, I would at least suggest using : or - plus a more specific keyword after it to make it more specific, like many of Keras's labels for example.

yishn commented 5 years ago

I'm keeping the "enhancement" label. Please stop posting off-topic comments.