15148 add copy button to config context #15954
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
arthanson
changed the title
jeremystretch
requested changes
May 6, 2024
There was a problem hiding this comment.
Can we refactor the inclusion templates into one? It seems very fragile to have the logic split apart like this. We should be able to move the content from configcontext_format.html into configcontext_data.html and wrap it in a conditional that looks for include_header or similar.
jeremystretch
force-pushed
the
15148-copy-config-context
branch
from
f13c61a to
5b7c392
Compare
|
@arthanson I merged the content of |
|
@jeremystretch looks good. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes: #15148
Adds a copy button to the config context area - Added it to the panel title area next to the json/yaml format buttons (see screenshot)