Skip to content
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

[4.x] Fix JSON indenting #1049

Merged
merged 1 commit into from
Apr 9, 2021
Merged

[4.x] Fix JSON indenting #1049

merged 1 commit into from
Apr 9, 2021

Conversation

driesvints
Copy link
Member

@driesvints driesvints commented Apr 9, 2021

Fix indenting for JSON payloads. vue-json-pretty 1.7.0 introduced a breaking change on a minor release 😞 https://github.com/leezng/vue-json-pretty/releases/tag/v1.7.0

Fixes #1047

@taylorotwell taylorotwell merged commit 93d5c21 into 4.x Apr 9, 2021
@taylorotwell taylorotwell deleted the fix-json-indent branch April 9, 2021 12:46
mmachatschek added a commit to mmachatschek/horizon that referenced this pull request Feb 13, 2022
taylorotwell pushed a commit to laravel/horizon that referenced this pull request Feb 14, 2022
* Update bootstrap and remove jQuery and popper.js dependencies

Popper.js is needed for tooltips, using the browser native title attribute removes the need to keep this

* Update utility classes

https://getbootstrap.com/docs/5.1/migration/#utilities

* Fix badge color classes

https://getbootstrap.com/docs/5.1/migration/#badges

* Remove text decoration of anchor tags

https://getbootstrap.com/docs/5.1/migration/#content-reboot-etc

* Fix data-toggle attribute

https://getbootstrap.com/docs/5.1/migration/#javascript

* Remove text-decoration of route-link elements

https://getbootstrap.com/docs/5.1/migration/#content-reboot-etc

* Remove new border styling under thead

* Fix vue-json-pretty output with overflowing text

* Apply fixes for updated vue-json-pretty css

laravel/telescope#1049
mmachatschek added a commit to mmachatschek/horizon that referenced this pull request Mar 29, 2024
mmachatschek added a commit to mmachatschek/horizon that referenced this pull request Apr 4, 2024
taylorotwell added a commit to laravel/horizon that referenced this pull request Apr 8, 2024
* Update bootstrap and remove jQuery and popper.js dependencies

Popper.js is needed for tooltips, using the browser native title attribute removes the need to keep this

* Update utility classes

https://getbootstrap.com/docs/5.1/migration/#utilities

* Fix badge color classes

https://getbootstrap.com/docs/5.1/migration/#badges

* Remove text decoration of anchor tags

https://getbootstrap.com/docs/5.1/migration/#content-reboot-etc

* Fix data-toggle attribute

https://getbootstrap.com/docs/5.1/migration/#javascript

* Remove text-decoration of route-link elements

https://getbootstrap.com/docs/5.1/migration/#content-reboot-etc

* Remove new border styling under thead

* Fix vue-json-pretty output with overflowing text

* Apply fixes for updated vue-json-pretty css

laravel/telescope#1049

* utility changes (rebase changes)

* remove text-decoration-none

https://getbootstrap.com/docs/5.1/migration/#content-reboot-etc

* apply some fixes after visual comparison

* undo styling changes

* Update Alert.vue

---------

Co-authored-by: Taylor Otwell <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Payload, Session and Response Format
2 participants