Skip to content

feat: implement static loading placeholders for category sidebar#350

Open
pushtikadia wants to merge 2 commits into
ProdigyV21:mainfrom
pushtikadia:feat/sidebar-shimmer
Open

feat: implement static loading placeholders for category sidebar#350
pushtikadia wants to merge 2 commits into
ProdigyV21:mainfrom
pushtikadia:feat/sidebar-shimmer

Conversation

@pushtikadia

Copy link
Copy Markdown

… state closes #319

@ProdigyV21

Copy link
Copy Markdown
Owner

Thanks for this, the idea is good, but this PR is not ready yet.

After merging into current main, CategorySidebar.kt has broken leftover code around the new skeleton block, including }up, and the old SidebarRow arguments outside the block. That will not compile.

Also the title says animated skeleton, but right now it only adds static gray boxes. Please fix the structure so the original SidebarRow logic stays inside the else branch, remove the leftover comment/code, and either add real shimmer/animation or rename the PR to static loading placeholders.

@pushtikadia pushtikadia changed the title feat: implement animated layout skeleton for category sidebar loading… feat: implement animated layout skeleton... Jun 8, 2026
@pushtikadia pushtikadia changed the title feat: implement animated layout skeleton... feat: implement static loading placeholders for category sidebar Jun 8, 2026
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

Successfully merging this pull request may close these issues.

[UI/UX]: Add Custom Shimmer/Loading Placeholder for IPTV Category List

2 participants