Skip to content

fix: Make QariList BottomSheetScaffold container color transparent#3396

Merged
ahmedre merged 1 commit intoquran:mainfrom
ahmed-okkio:main
Oct 15, 2025
Merged

fix: Make QariList BottomSheetScaffold container color transparent#3396
ahmedre merged 1 commit intoquran:mainfrom
ahmed-okkio:main

Conversation

@ahmed-okkio
Copy link
Copy Markdown
Contributor

@ahmed-okkio ahmed-okkio commented Oct 14, 2025 •
edited
Loading

After the dependency upgrade here #3388, the Qari list started leaving the container background when closed. This change fixed it for me but I'm unsure why this happened so this might not be the best solution

image image

@ahmedre
Copy link
Copy Markdown
Contributor

ahmedre commented Oct 15, 2025

JazakumAllah khairan - saw this in another project also and fixed it the same way

@ahmedre ahmedre enabled auto-merge October 15, 2025 01:17
@ahmedre ahmedre added this pull request to the merge queue Oct 15, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 15, 2025 

OLD: app-madani-debug.apk (signature: V1, V2)
NEW: app-madani-debug.apk (signature: V1, V2)

          │          compressed           │         uncompressed          
          ├───────────┬───────────┬───────┼───────────┬───────────┬───────
 APK      │ old       │ new       │ diff  │ old       │ new       │ diff  
──────────┼───────────┼───────────┼───────┼───────────┼───────────┼───────
      dex │  19.7 MiB │  19.7 MiB │ +33 B │  49.4 MiB │  49.4 MiB │ +24 B 
     arsc │   2.1 MiB │   2.1 MiB │   0 B │   2.1 MiB │   2.1 MiB │   0 B 
 manifest │   6.1 KiB │   6.1 KiB │   0 B │  29.9 KiB │  29.9 KiB │   0 B 
      res │   1.8 MiB │   1.8 MiB │  -2 B │     2 MiB │     2 MiB │   0 B 
   native │  18.8 KiB │  18.8 KiB │   0 B │  36.5 KiB │  36.5 KiB │   0 B 
    asset │ 452.1 KiB │ 452.1 KiB │   0 B │ 808.2 KiB │ 808.2 KiB │   0 B 
    other │ 198.1 KiB │ 198.1 KiB │   0 B │   495 KiB │   495 KiB │   0 B 
──────────┼───────────┼───────────┼───────┼───────────┼───────────┼───────
    total │  24.2 MiB │  24.2 MiB │ +31 B │  54.9 MiB │  54.9 MiB │ +24 B

Merged via the queue into quran:main with commit a45e65b Oct 15, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ahmed-okkio @ahmedre