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

[all] create name convention for layout IDs #369

Open
mtrakal opened this issue Jul 24, 2017 · 0 comments
Open

[all] create name convention for layout IDs #369

mtrakal opened this issue Jul 24, 2017 · 0 comments

Comments

@mtrakal
Copy link

mtrakal commented Jul 24, 2017

There is no convention for layout view ids!
In your sample code are somerimes camelCase, sometimes snake_case underscope ids. It make a mess. So please use only one and everywhere same name convention.

For me is prefered camelCase (due to kotlinx) with some prefix like v (view).

What is Google's prefered code style and why developers in Google don't use same style? :/

https://github.com/googlesamples/android-architecture/blob/todo-mvp-dagger/todoapp/app/src/main/res/layout/addtask_act.xml#L45
android:id="@+id/coordinatorLayout"

https://github.com/googlesamples/android-architecture/blob/todo-mvp-dagger/todoapp/app/src/main/res/layout/addtask_frag.xml#L31
android:id="@+id/add_task_title"

@mtrakal mtrakal changed the title Please create name convention for layout IDs [all] create name convention for layout IDs Jul 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant