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

Android: remove save icon and add up button #10358

Merged
merged 1 commit into from Jan 8, 2022

Conversation

Simonx22
Copy link
Member

@Simonx22 Simonx22 commented Jan 7, 2022

This PR removes the save icon since it's no longer necessary. I've also added an up button to the top left of the screen in the settings and cheats view, which makes navigating feel much more natural.

Screenshots:

@JosJuice
Copy link
Member

JosJuice commented Jan 7, 2022

Could you call this an up button instead of a back button? Back and up are not the same thing.

@Simonx22 Simonx22 force-pushed the android-back-button branch 2 times, most recently from 516353d to b2cd25b Compare January 7, 2022 18:27
@Simonx22 Simonx22 changed the title Android: remove save icon and add back button Android: remove save icon and add up button Jan 7, 2022
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.

I'm not sure if calling onBackPressed is a standard way to implement the Up button, but regardless, this brings our UI more in line with what Android apps normally are like. LGTM.

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.

Oops, I meant to click Approve and not Comment.

@dolphin-emu-bot
Copy link
Contributor

FifoCI detected that this change impacts graphical rendering. Here are the behavior differences detected by the system:

  • aeon-charge-attack on mvk-osx-m1: diff
  • burnout2-vehicletextures on mvk-osx-m1: diff
  • DKCR-fast-depth on mvk-osx-m1: diff
  • ea-pink on mvk-osx-m1: diff
  • mp2-scanner on mvk-osx-m1: diff
  • rs2-glass on mvk-osx-m1: diff
  • spyro-bloom on mvk-osx-m1: diff
  • thps4-shadow on mvk-osx-m1: diff

automated-fifoci-reporter

@JMC47 JMC47 merged commit 94ad33c into dolphin-emu:master Jan 8, 2022
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants