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

Back button call menu in ButtonMapping Activities #511

Merged
merged 2 commits into from
Jul 11, 2015

Conversation

BlisterB
Copy link
Member

@BlisterB BlisterB commented Jul 5, 2015

No description provided.

@BlisterB
Copy link
Member Author

BlisterB commented Jul 5, 2015

I remembered that in Player Activity, back button ask a confirmation to quit the game.

Should I delete this behavior to call the menu too ?
Pro :

@fdelapena
Copy link
Contributor

Because some recent devices don't have hardware menu buttons (only home and back buttons), I think any hardware involved menu button should be removed. Adding an action bar for this and stealing space in the game screen is ugly so this feature could be added to the back button instead.

A good device example without this button: HTC One.

@BlisterB
Copy link
Member Author

BlisterB commented Jul 5, 2015

@fdelapena there is no hardware button involved in the android port :p . (My phone doesn't have such a button and I have no problem).
When we associate a menu to an activity, it creates an hamburger icon in the dock (software button) to reach the menu .

But I think I'll associate the menu to back button to the Player Activity, because back button is already associated to a pop-up box (asking for stopping the game).

@BlisterB
Copy link
Member Author

BlisterB commented Jul 5, 2015

Done !

@Ghabry
Copy link
Member

Ghabry commented Jul 5, 2015

Thanks for the fix. Just got back from a short weekend trip. Will check tomorrow :)

@fdelapena
Copy link
Contributor

Tested here, back button works for adding/editing from the screen button layout editor with latest pull request generated .apk 👍.

@Ghabry
Copy link
Member

Ghabry commented Jul 11, 2015

Sorry that this took so long, was without internet the last days.
Works perfect, thx.

Ghabry added a commit that referenced this pull request Jul 11, 2015
Back button call menu in ButtonMapping Activities
@Ghabry Ghabry merged commit 5aeea88 into EasyRPG:master Jul 11, 2015
@Ghabry
Copy link
Member

Ghabry commented Jul 11, 2015

One issue is still that the config function of the Game Browser is not obvious.
Similiar for long press to select region.

@fdelapena
Copy link
Contributor

These UX issues have been reported at #512 some days ago.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants