Opened 6 years ago

Closed 6 years ago

#10435 closed feature request (fixed)

Shown unknown game variant message in GUI

Reported by: criezy Owned by: rootfather
Priority: normal Component: GUI
Version: Keywords:
Cc: Game:

Description

Currently when an unknown game variant is detected, it adds a message in the log file and stdout so that the user can easily copy it to the clipboard and report the unknown variant. However this makes the message difficult to notice on some platforms. I think it should also be shown in the GUI in a dialog. And to make it easy to report the unknown game variant, this dialog could have, depending on the features available for the current platform:

  • A button to copy the message to the clipboard.
  • A button to open the bug tracker and pre-fill the description.

We already have an API to open a URL in OSystem, but the copy to clipboard is currently missing (we have only a paste from clipboard) and would need to be added.

Change History (8)

comment:1 by criezy, 6 years ago

To clarify, the message I am thinking of here is the one in AdvancedMetaEngine::reportUnknown() in engines/advancedDetector.cpp. There is also a very similar message in ScummMetaEngine::createInstance() in engines/scumm/detection.cpp.

comment:2 by rootfather, 6 years ago

Currently working on the dialog and clipboard part. First results may follow tomorrow.

comment:3 by rootfather, 6 years ago

Owner: set to rootfather

comment:4 by criezy, 6 years ago

rootfather: I actually started working on that yesterday and implemented the clipboard part but haven't looked at the dialog at all, so you are welcome to work on this (otherwise I was planning to have a look next week-end).

I just pushed the clipboard commits to my fork in case you want to reuse them:
https://github.com/scummvm/scummvm/compare/master...criezy:unknown-game-dialog

comment:5 by rootfather, 6 years ago

criezy: Your implementation is far ahead of what I achieved so far, you implemented the clipboard part in a much nicer way than I did. I don't know if it makes sense to work on the dialog part or better leave this to you too...

comment:6 by criezy, 6 years ago

rootfather: it's really up to you. If you want to work on this dialog, this could be a relatively easy task to get some more coding experience, and I will be happy to help you or give you feedback if needed. On the other hand if you prefer me to do it, I can work on it this weekend as I initially planned.

comment:7 by rootfather, 6 years ago

criezy: Thanks for offering me help and feedback, I really appreciate it. I think I'll tackle this task then. Code will live at https://github.com/scummvm/scummvm/compare/master...rootfather:unknown-game-dialog.

comment:8 by criezy, 6 years ago

Resolution: fixed
Status: newclosed

The pull request from rootfather to add this dialog has now been merged.
Note that currently only the AdvancedMetaEngine uses this dialog. It would be good for other engines to also use it.

Note: See TracTickets for help on using tickets.