-
Notifications
You must be signed in to change notification settings - Fork 50
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
Removed about and settings from the top right #56
Conversation
Removed about and settings from the top right of the menu.
Thank you for the pull request!The Scribe team will do our best to address your contribution as soon as we can. The following is a checklist for maintainers to make sure this process goes as well as possible. Feel free to address the points below yourself in further commits if you realize that actions are needed :) If you're not already a member of our public Matrix community, please consider joining! We'd suggest using Element as your Matrix client, and definitely join the General and Android rooms once you're in. Also consider joining our bi-weekly Saturday dev syncs. It'd be great to have you! Maintainer checklist
|
@@ -116,7 +116,7 @@ class MainActivity : SimpleActivity(), GestureDetector.OnGestureListener { | |||
override fun onOptionsItemSelected(item: MenuItem): Boolean { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Scribe-Android/app/src/main/kotlin/be/scri/activities/MainActivity.kt
Lines 110 to 124 in abfea22
override fun onCreateOptionsMenu(menu: Menu): Boolean { | |
menuInflater.inflate(R.menu.menu_main, menu) | |
updateMenuItemColors(menu) | |
return true | |
} | |
override fun onOptionsItemSelected(item: MenuItem): Boolean { | |
when (item.itemId) { | |
R.id.settings -> launchSettings() | |
R.id.info -> launchAbout() | |
else -> return super.onOptionsItemSelected(item) | |
} | |
return true | |
} | |
These two functions can be removed as they are no longer required since the top menu has been removed.
app/src/main/res/menu/menu_main.xml
Outdated
@@ -1,14 +1,3 @@ | |||
<?xml version="1.0" encoding="utf-8"?> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file could be removed as it is no longer required. and as the references of id have been renamed to R.id.info.
Thanks for the review, @angrezichatterbox! Let us know if you have any questions, @henrikth93 :) |
Removed onCreateOptionsMenu and onOptionsItemSelected. ALso removed menu_main.xml since it is not needed anymore.
Updated the code now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Welcome to the Scribe-Android contributors, @henrikth93! 🎉 Thanks so much for this,a and looking forward to the next PR where we get the Scribe logo in! Will start feeling a lot more familiar then 😊
Removed about and settings from the top right of the menu.
Contributor checklist
Description
Related issue