Skip to content

Update README screenshot to match current UI#194

Merged
jbampton merged 10 commits into
john-bampton:mainfrom
MalathiBalaraman31:fix-center-loading
May 1, 2026
Merged

Update README screenshot to match current UI#194
jbampton merged 10 commits into
john-bampton:mainfrom
MalathiBalaraman31:fix-center-loading

Conversation

@MalathiBalaraman31

Copy link
Copy Markdown
Contributor

Updates the README screenshot to reflect the current UI shown after a hard refresh.

Fixes #191

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@deepsource-io

deepsource-io Bot commented Dec 26, 2025

Copy link
Copy Markdown
Contributor

DeepSource Code Review

We reviewed changes in 1f26b84...a58b060 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
Secrets May 1, 2026 3:00a.m. Review ↗
Python May 1, 2026 3:00a.m. Review ↗
JavaScript May 1, 2026 3:00a.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@BaseMax

BaseMax commented Dec 26, 2025

Copy link
Copy Markdown
Member

Hi there, first of all many thanks for your help.

But, 1- Please do it again as we merged a new nice feature recently at #187
2- If it's possible please try on another browser as your scrollbar is not as nice as previus screenshot. I not sure what is your browser.
3- Important point: if you pay attention to filename you can see filename is screenshot-1280.png‎ so width of the screenshot and image should be excatly 1280.
But your screenshot size is W: 1903px | H: 907px.

@BaseMax

BaseMax commented Dec 26, 2025

Copy link
Copy Markdown
Member

4- It seems there is a tooltip on your screen, you need to get screenshot in a normal state without any tooltip or hover. such as:
image

@BaseMax

BaseMax commented Dec 26, 2025

Copy link
Copy Markdown
Member

5- Make sure there is no link at bottom (left or right) of your browser such as:

image

@BaseMax

BaseMax commented Dec 26, 2025

Copy link
Copy Markdown
Member

Here is a screenshot that you can use:

image

Just to add, it will be great if we shows 4 columns in width 1280px. currently we are showing 3 columns and this is not great.

@BaseMax

BaseMax commented Dec 26, 2025

Copy link
Copy Markdown
Member

#196 created

@BaseMax

BaseMax commented Dec 26, 2025

Copy link
Copy Markdown
Member

Just do and finalize this PR, after solving the #196.

Many thanks.

@MalathiBalaraman31

MalathiBalaraman31 commented Dec 26, 2025

Copy link
Copy Markdown
Contributor Author

@BaseMax Got it, thanks for the clarification
I’ll work on #196 first and then finalize this PR.

@MalathiBalaraman31

Copy link
Copy Markdown
Contributor Author

@BaseMax Issue #196 has been fixed and the README screenshot has been updated to exact 1280px width (4 columns, no hover/UI artifacts). Ready for final review. Thanks!

@BaseMax

BaseMax commented Dec 26, 2025

Copy link
Copy Markdown
Member

cc #196 (comment)

@BaseMax

BaseMax commented Dec 26, 2025

Copy link
Copy Markdown
Member

Please update another screenshot file as well. with a diffrent width, not 1280px.

@BaseMax BaseMax left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jbampton jbampton self-assigned this Dec 26, 2025
@jbampton jbampton added the documentation Improvements or additions to documentation label Dec 26, 2025
@jbampton jbampton added this to Turbo Dec 26, 2025
@jbampton jbampton added this to the December - January 2025 milestone Dec 26, 2025
@jbampton jbampton moved this to In Progress in Turbo Dec 26, 2025
@BaseMax

BaseMax commented Dec 26, 2025

Copy link
Copy Markdown
Member

Please update another screenshot file as well. with a diffrent width, not 1280px.

please check other screenshot images on the repo dir.

@MalathiBalaraman31

Copy link
Copy Markdown
Contributor Author

Updated the 540px screenshot as requested. Thanks!

@jbampton jbampton self-requested a review as a code owner May 1, 2026 02:49

@jbampton jbampton left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks !!

@jbampton jbampton merged commit 8400735 into john-bampton:main May 1, 2026
5 of 6 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Turbo May 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Super easy task: update the screenshot on README.md

3 participants