-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Replace images by more recent one #12628
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
Conversation
Thank you very much! Can you please update the exception image including the toolbar like before and try to decrease the size? ist ~260 kb vs. 76 kb before |
@OskarStark I will do it, but maybe should I wait until sf 4.4 lts is out, so will be more "stable" images? |
No let’s use them for 4.3 now and maybe update them later for 4.4 |
1b4c182
to
f27beb9
Compare
@OskarStark here you go note that i had to create a project with images are also compressed a little bit as requested cheers |
a1503bf
to
285b832
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! I love contributions like this: It seems small, but it makes a documentation much better if the screenshots are up to date :)
285b832
to
26763ac
Compare
This PR was submitted for the master branch but it was merged into the 4.3 branch instead (closes #12628). Discussion ---------- Replace images by more recent one Just more recent images (I was testing the 4.3 branch) I will do the same when 4.4 will be released Commits ------- 26763ac Replace images by more recent one
Thank you 👍 Would love to receive a PR for |
Just more recent images (I was testing the 4.3 branch)
I will do the same when 4.4 will be released