Commit 45d36cc
Bug fixes and improvements following initial review
I'm glad you like the design! :) I tried to address all the issues you mentioned:
**UI Changes:**
- Switched the **collapsing toolbar layout** for a standard TopAppBar/back to the action bar
- Improved **title display** when clicking an alert
- Fixed the **map showing cropped**
- Fixed the **share button stuttering** when switching to map tab for a moment
**Bug Fixes:**
- Fixed the **map only loading on automatic map reload** and taking 10 seconds to load
- Got **share button** back
- Fixed the **restore button not showing** on the alerts list after clearing all alerts
- Fixed the **"Select your city" popup** to redirect to the actual city selection dialog
**Improvements:**
- Added back a **loading indicator** on automatic map refreshes
- Added an **animation for clearing and restoring** the recent alerts list :)
- Implemented **asynchronous loading** for the map when going to the map tab (preventing delay when switching to it)
- Added **predictive back animation** when switching between settings screens
- Made the action bar title **immediately change back** to recent alerts when going back from an alert rather than waiting for animation to finish to make the app feel more responsive
**Other:**
- Updated to **TargetSdkVersion 35**
I couldn't seem to reproduce the 2 clear recent alerts buttons appearing or the back button not working when viewing an individual alert - could you please check if it's working as expected for you now? :)1 parent 0978a60 commit 45d36cc
146 files changed
Lines changed: 7480 additions & 3244 deletions
File tree
- app
- src/main
- java/com/red/alert
- activities
- settings
- alerts
- config
- logic/settings
- model
- ui
- activities
- adapters
- compatibility
- dialogs
- custom
- elements
- dialogs
- fragments
- utils
- localization
- metadata
- ui
- res
- animator
- anim
- color
- drawable
- layout
- values-ar
- values-de
- values-en
- values-es
- values-fr
- values-it
- values-night
- values-pt
- values-ru
- values
- xml
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
13 | | - | |
14 | | - | |
| 13 | + | |
| 14 | + | |
15 | 15 | | |
16 | 16 | | |
17 | 17 | | |
| |||
29 | 29 | | |
30 | 30 | | |
31 | 31 | | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
32 | 49 | | |
33 | 50 | | |
34 | 51 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
| 11 | + | |
11 | 12 | | |
12 | 13 | | |
13 | 14 | | |
| |||
38 | 39 | | |
39 | 40 | | |
40 | 41 | | |
| 42 | + | |
41 | 43 | | |
42 | 44 | | |
43 | 45 | | |
| |||
0 commit comments