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

MemoryWidget: Split Actions into Import and Export. #11250

Merged

Conversation

AdmiralCurtiss
Copy link
Contributor

I've seen multiple people not realize that the Actions is a menu that's clickable. Does this improve the situation, by making a second menu option which makes it clearer that it's a menu bar?

actions
importexport

@jordan-woyak
Copy link
Member

It's better, but it's still an unusual and unexpected place for a menu bar I think.

Copy link
Member

@JosJuice JosJuice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Even if we manage to think of something better later, let's merge this for now.

@AdmiralCurtiss AdmiralCurtiss merged commit c75b53c into dolphin-emu:master Nov 6, 2022
11 checks passed
@AdmiralCurtiss AdmiralCurtiss deleted the memory-widget-menu branch November 6, 2022 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants