android / android/add-adaptive-layouts
Please update to mention updating hte w600dp/activity_main.xml
- Dominant language
- Kotlin
- Stars
- 23
- Forks
- 12
- PR merge metrics
- No merged PRs in 30d
Description
In the **Replace the BottomNavigationView element with the ComposeView element in the layout resource file** section.
It currently says:
> In the activity_main.xml file, replace the BottomNavigationView element with the ComposeView element. This embeds the bottom navigation component into a view-based UI layout with Compose.
However it doesn't make any mention of the `w600dp/activity_main.xml` file which will also need to be updated if this is going to work on Tablets and for the code to properly run.
I recommend adding a mention in that section about updating both files. Thanks!
Contributor guide
Research direction
Find the documentation section titled "Replace the BottomNavigationView element with the ComposeView element in the layout resource file" and review its references to activity_main.xml. Update the instructions to mention both activity_main.xml and w600dp/activity_main.xml, including the tablet layout, then verify the section clearly describes updating both files.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- android, kotlin
- Domain
- documentation
- Issue type
- Documentation
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 55/100