Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make the "Settings" folder more obvious and clickable #2606

Merged

Conversation

KABoissonneault
Copy link
Collaborator

Trying to debug any user issue with DFU or a mod usually runs into this conversation.

"Hey, my game is broken."
"Can you post logs?"
"Where?"
"... Wait a second" searches a bit "Should be in C:\<User>\AppData\LocalLow\Daggerfall Workshop\Daggerfall Unity, replace <User> with your Windows username. Or you can try copy-pasting %USERPROFILE%\AppData\LocalLow\Daggerfall Workshop\Daggerfall Unity in your Explorer. Then..."
user disappears

With this change, I'm trying to reduce it to:
"Open DFU, double-click on your Settings Folder, copy-paste Player.log"

This would apply for all other settings and mod data concerns.

Current Title Screen for comparison:
image

New Title Screen:
image

New Title Screen, with Hide Login Name setting enabled:
image-1

I wanted to add "Settings Folder" to emphasize what this path was. I couldn't put it on the left, because localization would make the alignment a mess for this screen (mods can't move labels on this screen). But I was running out of vertical space, so I move things around a bit.

The path has a tooltip telling you to double-click, and double-clicking opens the folder. I avoided going to single click, since it's in the middle of the screen and easy to misclick.

I tested on all aspect ratios on Unity, no issues.

… and allow double-clicking to open the folder
@KABoissonneault KABoissonneault added this to the DFU 1.0.1 milestone Mar 1, 2024
@KABoissonneault KABoissonneault mentioned this pull request Mar 3, 2024
@KABoissonneault KABoissonneault merged commit 184fc25 into Interkarma:master Mar 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants