Removed about and settings from the top right#56
Removed about and settings from the top right#56andrewtavis merged 2 commits intoscribe-org:mainfrom
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.
Scribe-Android/app/src/main/kotlin/be/scri/activities/MainActivity.kt
Lines 110 to 124 in abfea22
These two functions can be removed as they are no longer required since the top menu has been removed.
| @@ -1,14 +1,3 @@ | |||
| <?xml version="1.0" encoding="utf-8"?> | |||
There was a problem hiding this comment.
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. |
andrewtavis
left a comment
There was a problem hiding this comment.
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