Skip to content

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

Merged
merged 1 commit into from
Nov 19, 2019

Conversation

noniagriconomie
Copy link
Contributor

@noniagriconomie noniagriconomie commented Nov 12, 2019

Just more recent images (I was testing the 4.3 branch)
I will do the same when 4.4 will be released

@OskarStark
Copy link
Contributor

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

cc @javiereguiluz

@OskarStark OskarStark changed the base branch from master to 4.3 November 12, 2019 12:54
@OskarStark OskarStark changed the base branch from 4.3 to master November 12, 2019 12:54
@OskarStark OskarStark added this to the 4.3 milestone Nov 12, 2019
@noniagriconomie
Copy link
Contributor Author

@OskarStark I will do it, but maybe should I wait until sf 4.4 lts is out, so will be more "stable" images?

@OskarStark
Copy link
Contributor

No let’s use them for 4.3 now and maybe update them later for 4.4

@noniagriconomie
Copy link
Contributor Author

noniagriconomie commented Nov 18, 2019

@OskarStark here you go

note that i had to create a project with symfony new sf43 --version=4.3 --full --debug (note the --full) to have the wdt (i do not know if we should document it or not)

images are also compressed a little bit as requested

cheers

@noniagriconomie noniagriconomie changed the base branch from master to 4.3 November 18, 2019 17:15
@noniagriconomie noniagriconomie changed the base branch from 4.3 to master November 18, 2019 17:15
@noniagriconomie noniagriconomie force-pushed the error_pages_images branch 2 times, most recently from a1503bf to 285b832 Compare November 18, 2019 17:18
Copy link
Member

@wouterj wouterj left a 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 :)

@OskarStark OskarStark changed the base branch from master to 4.3 November 19, 2019 07:35
OskarStark added a commit that referenced this pull request Nov 19, 2019
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
@OskarStark
Copy link
Contributor

Thank you 👍

Would love to receive a PR for 4.4 (and 5.0, if there are any differences) 💯

@OskarStark OskarStark merged commit 26763ac into symfony:4.3 Nov 19, 2019
@noniagriconomie noniagriconomie deleted the error_pages_images branch November 19, 2019 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants