feat(ui): add session-based screenshot panel with thumbnails#87
feat(ui): add session-based screenshot panel with thumbnails#87MadhuTiwari-345 wants to merge 10 commits intoHarshYadav152:mainfrom
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
🎉 Thank you @MadhuTiwari-345 for your first PR to SS-Capture!
We really appreciate your contribution 🙌
What happens next:
- 🔍 Maintainers will review your PR
- 🧪 Automated checks will run
- ✨ Feedback may be shared if needed
Please confirm your PR includes:
- ✔️ Clear summary of changes
- ✔️ Linked issue (e.g., Fixes #123)
- ✔️ Steps to test
- ✔️ Screenshots (for UI changes)
📘 Contribution Standards:
👉 https://github.com/HarshYadav152/ss-capture/blob/main/CONTRIBUTING.md
💬 Stay Connected with Our Community
🎯 Discord (Official Communication)
For formal discussions, code reviews, and project updates:
👉 https://discord.gg/eMTePUK3gJ
💚 WhatsApp (Friendly Community)
For informal chats, quick help, and building friendships with contributors:
👉 https://chat.whatsapp.com/I8GYXd3mHlDCC2iXhNGeqV
🌟 Our Philosophy: We value both professional collaboration (Discord) and personal connections (WhatsApp). Join both to get the complete SS-Capture community experience!
Thanks for helping improve SS-Capture 🚀
Let's build something amazing together! 💪
HarshYadav152
left a comment
HarshYadav152
left a comment
There was a problem hiding this comment.
- Remove
.gitignorefrom the root. - Do not modified
CHANGELOG.mdfile as it is auto modified byauto/changelogauto PR. - Your recent commits break the previous functionality.
- Also theme switching mode not working.
- There are some issues in
src/js/content.js. - Recheck your
manifest.jsonat thesrc/.
- Keep you session related
css, jsfiles at inside a one folder insidesrc/css/a session-history directory or whatever you prefered. - And remove workflow you have been added
ci.yml.
test your changes with all previous features/functionality.
and one most important thing keep your branch is up to date with upstream main.
MadhuTiwari-345
force-pushed
the
feature/screenshot-panel
branch
from
5771b86 to
cf60742
Compare
MadhuTiwari-345
force-pushed
the
feature/screenshot-panel
branch
from
cf60742 to
8dce721
Compare
HarshYadav152
left a comment
HarshYadav152
left a comment
There was a problem hiding this comment.
- remove
ss-capture/.github/workflows/ci.yml - do not modified
CHANGELOG.md
and there are some error in manifest file so extension will stuck in loading phase and never reaches captured ss.
|
Hi @HarshYadav152 , I’ve, Removed ss-capture/.github/workflows/ci.yml, |
|
@MadhuTiwari-345 Still have same problem it has been stuck while capturing screen shot and never reaches the end. |
|
@MadhuTiwari-345 Complete it ASAP. |
|
Sorry for the inconvenience
I'm working on it right now
By the evening it might be done.
I'm sorry again
…On Wed, Jan 14, 2026, 3:52 PM Harsh Yadav ***@***.***> wrote:
*HarshYadav152* left a comment (HarshYadav152/ss-capture#87)
<#87 (comment)>
@MadhuTiwari-345 <https://github.com/MadhuTiwari-345> Complete it ASAP.
—
Reply to this email directly, view it on GitHub
<#87 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BONO2UF4TE4JG7WHFOWXRSL4GYKFNAVCNFSM6AAAAACQ6KL3UKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTONBYHAZDOOJVGU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Be active nd responsive on your every PR. |
|
Hi @HarshYadav152 I'm sorry, I'm still working on it. It has a lot of bugs 
s
|
|
@MadhuTiwari-345 Alright take your time but stay responsive and active. |
HarshYadav152
left a comment
HarshYadav152
left a comment
There was a problem hiding this comment.
What is the progress?
Also keep your branch with upstream main.
|
Yeah sure I'm keeping it upstream I fixed all the bugs and extension is
otherwise fine when I'm testing on terminal without errors but when I'm
manually testing capture kept getting failed
…On Sat, Jan 17, 2026, 11:07 AM Harsh Yadav ***@***.***> wrote:
***@***.**** requested changes on this pull request.
What is the progress?
Also keep your branch with upstream main.
—
Reply to this email directly, view it on GitHub
<#87 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BONO2UHBJDJHKMG6L5YLG4D4HHDB5AVCNFSM6AAAAACQ6KL3UKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZTMNZTGUYDCMBXG4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
MadhuTiwari-345
left a comment
MadhuTiwari-345
left a comment
There was a problem hiding this comment.
Hi @HarshYadav152 Please check it out I have tested the extension it is working well It is capturing the image and we can save the screenshot by the option save screenshot & copy the image by copy the image option
HarshYadav152
left a comment
HarshYadav152
left a comment
There was a problem hiding this comment.
- Current changes did something bad with opened tab
- When capturing ss it will give an error
Failed to fetch html2canvas script: Failed to fetch
fix these and keep your branch updated with main
|
okay I'll work on it..actually there was another file to I think I didn't
commit and pushed it thats why it is showing this error
When I took screenshot it worked well I even saved it in my gallery
…On Tue, Jan 20, 2026, 4:43 PM Harsh Yadav ***@***.***> wrote:
***@***.**** requested changes on this pull request.
1. Current changes did something bad with opened tab
image.png (view on web)
<https://github.com/user-attachments/assets/c2becb74-a73c-4fd6-8a51-0263c28b39ee>
2. When capturing ss it will give an error Failed to fetch html2canvas
script: Failed to fetch
fix these and keep your branch updated with main
—
Reply to this email directly, view it on GitHub
<#87 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BONO2UCNSIK7FFZHHHAEJIT4HYEU3AVCNFSM6AAAAACQ6KL3UKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZTMOBRGY3DGMBTGY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
Is that done ? |
|
hi its not done yet..actually I have an exam on 24 Jan can you give me 2
days?..Actually that day I didn't commit html2canvas script thats why the
error might be happening
…On Wed, Jan 21, 2026, 7:28 PM Harsh Yadav ***@***.***> wrote:
*HarshYadav152* left a comment (HarshYadav152/ss-capture#87)
<#87 (comment)>
Is that done ?
—
Reply to this email directly, view it on GitHub
<#87 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BONO2UGLULCP2WJRO3JLLM34H6AZRAVCNFSM6AAAAACQ6KL3UKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTONZYGI3TINJQGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Alright. |
Alright. |
|
okay👍
…On Wed, Jan 21, 2026, 9:37 PM Harsh Yadav ***@***.***> wrote:
*HarshYadav152* left a comment (HarshYadav152/ss-capture#87)
<#87 (comment)>
hi its not done yet..actually I have an exam on 24 Jan can you give me 2
days?..Actually that day I didn't commit html2canvas script thats why the
error might be happening
On Wed, Jan 21, 2026, 7:28 PM Harsh Yadav *@*.***> wrote:
*HarshYadav152* left a comment (#87
<#87>)
#87 (comment)
<#87 (comment)>
Is that done ?
—
Reply to this email directly, view it on GitHub
#87 (comment)
<#87 (comment)>
,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BONO2UGLULCP2WJRO3JLLM34H6AZRAVCNFSM6AAAAACQ6KL3UKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTONZYGI3TINJQGI
.
You are receiving this because you were mentioned.Message ID:
*@*.***>
Alright.
But be responsive
—
Reply to this email directly, view it on GitHub
<#87 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BONO2UHON6S7A6735NTECZL4H6P3NAVCNFSM6AAAAACQ6KL3UKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTONZZGIYTGMZWGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
Hi @HarshYadav152, Please Check it out |
2 participants
Description
This PR adds a session-based screenshot panel that enhances how users manage and access screenshots during a single browsing session.
It introduces a lightweight, collapsible UI panel that displays thumbnails of all screenshots captured in the current session. Each thumbnail allows users to quickly preview or download the screenshot without leaving the extension workflow.
Type of Change
What this PR does
Why this change is needed
Implementation Details
Testing
Checklist
Related Issue
Closes #27