-
Notifications
You must be signed in to change notification settings - Fork 607
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Show details by default #1519
Comments
@mikestreety are you looking for this? |
🤦 Yes. Apparently so. Is there anyway of having this on by default? |
I'm not sure, but it's a great idea for a feature! |
dgrebb
added a commit
to dgrebb/BackstopJS
that referenced
this issue
Jan 17, 2024
dgrebb
added a commit
to dgrebb/BackstopJS
that referenced
this issue
Jan 18, 2024
dgrebb
added a commit
to dgrebb/BackstopJS
that referenced
this issue
Jan 18, 2024
dgrebb
added a commit
to dgrebb/BackstopJS
that referenced
this issue
Jan 18, 2024
dgrebb
added a commit
to dgrebb/BackstopJS
that referenced
this issue
Jan 18, 2024
dgrebb
added a commit
to dgrebb/BackstopJS
that referenced
this issue
Jan 18, 2024
@mikestreety PR #1538 is open for this. If you'd like, have a peek by installing the branch: npm install dgrebb/BackstopJS#feat/1519-suiteInfo-localStorage-hydration Thanks again for the feedback! |
garris
pushed a commit
that referenced
this issue
Feb 1, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hey, thanks for a great tool :)
I was wondering if you would consider showing all the information by default rather than having it hidden? I appreciate the clean interface, but I didn't notice that the information was hidden until I went to click to copy the file name.
I'm not so confident in getting this spun up to make an MR, but I think the changes would be:
Change this styling to
And then remove these lines and everything pertaining to
settings.textInfo
The text was updated successfully, but these errors were encountered: