Skip to content

Conversation

@MalathiBalaraman31
Copy link
Contributor

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

Fixes #191

@gemini-code-assist
Copy link
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
Copy link
Contributor

deepsource-io bot commented Dec 26, 2025

Here's the code health analysis summary for commits 9c0d869..f40cf6f. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Secrets LogoSecrets✅ SuccessView Check ↗
DeepSource Python LogoPython✅ SuccessView Check ↗
DeepSource JavaScript LogoJavaScript✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@BaseMax
Copy link
Member

BaseMax commented Dec 26, 2025

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
Copy link
Member

BaseMax commented Dec 26, 2025

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
Copy link
Member

BaseMax commented Dec 26, 2025

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

image

@BaseMax
Copy link
Member

BaseMax commented Dec 26, 2025

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
Copy link
Member

BaseMax commented Dec 26, 2025

#196 created

@BaseMax
Copy link
Member

BaseMax commented Dec 26, 2025

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

Many thanks.

@MalathiBalaraman31
Copy link
Contributor Author

MalathiBalaraman31 commented Dec 26, 2025

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

@MalathiBalaraman31
Copy link
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
Copy link
Member

BaseMax commented Dec 26, 2025

cc #196 (comment)

@BaseMax
Copy link
Member

BaseMax commented Dec 26, 2025

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

Copy link
Member

@BaseMax BaseMax left a comment

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
Copy link
Member

BaseMax commented Dec 26, 2025

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
Contributor Author

Updated the 540px screenshot as requested. Thanks!

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: In Progress

Development

Successfully merging this pull request may close these issues.

Super easy task: update the screenshot on README.md

3 participants