-
-
Notifications
You must be signed in to change notification settings - Fork 528
Deprecate View Based UI
#1067
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
Deprecate View Based UI
#1067
Conversation
- move sample app classes to kotlin source folder
- update to kotlinX serialization 1.8.0 - cleanup
…deprecated view based UI
| <activity | ||
| android:name="com.mikepenz.aboutlibraries.sample.ComposeActivity" | ||
| android:exported="true" | ||
| android:label="@string/app_name" |
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.
| Redundant label can be removed |
| android:theme="@style/SampleApp.DayNight" /> | ||
| android:name="com.mikepenz.aboutlibraries.sample.FragmentActivity" | ||
| android:exported="false" | ||
| android:label="@string/app_name" |
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.
| Redundant label can be removed |
| android:theme="@style/SampleApp.DayNight" /> | ||
| android:name="com.mikepenz.aboutlibraries.sample.ExtendActivity" | ||
| android:exported="false" | ||
| android:label="@string/app_name" |
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.
| Redundant label can be removed |
| android:theme="@style/SampleApp.DayNight" /> | ||
| android:name="com.mikepenz.aboutlibraries.sample.CustomSortActivity" | ||
| android:exported="false" | ||
| android:label="@string/app_name" |
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.
| Redundant label can be removed |
| import androidx.compose.foundation.layout.windowInsetsPadding | ||
| import androidx.compose.foundation.rememberScrollState | ||
| import androidx.compose.foundation.verticalScroll | ||
| import androidx.compose.material.BottomDrawer |
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.
| Using a material import while also using the material3 library |
| import androidx.compose.foundation.rememberScrollState | ||
| import androidx.compose.foundation.verticalScroll | ||
| import androidx.compose.material.BottomDrawer | ||
| import androidx.compose.material.BottomDrawerValue |
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.
| Using a material import while also using the material3 library |
| import androidx.compose.foundation.verticalScroll | ||
| import androidx.compose.material.BottomDrawer | ||
| import androidx.compose.material.BottomDrawerValue | ||
| import androidx.compose.material.Switch |
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.
| Using a material import while also using the material3 library |
| import androidx.compose.material.icons.filled.Menu | ||
| import androidx.compose.material.icons.filled.Person | ||
| import androidx.compose.material.icons.filled.Settings | ||
| import androidx.compose.material.rememberBottomDrawerState |
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.
| Using a material import while also using the material3 library |
Officially mark view-based UI as deprecated. In favor of the compose based UI.
Rework sample app to prioritize compose based UI and enhance it.