Update UI screenshots in Pipeline documentation #7526 #7653
+54
−48
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
ShrushtiDesai
force-pushed
the
pipeline-ss-alt-text-update
branch
from
bf77d24
to
e44ffa3
Compare
ShrushtiDesai
force-pushed
the
pipeline-ss-alt-text-update
branch
from
e44ffa3
to
f5f755f
Compare
ShrushtiDesai
force-pushed
the
pipeline-ss-alt-text-update
branch
from
f5f755f
to
5029a9f
Compare
kmartens27
approved these changes
Nov 4, 2024
There was a problem hiding this comment.
These all look really good, thanks very much for all the work @ShrushtiDesai
I've provided a few suggestions regarding the plugin macro usage and some punctuation, but everything else looks fantastic. Thank you as well for providing good alt text for the images. I'm approving, pending the changes suggested.
ShrushtiDesai
force-pushed
the
pipeline-ss-alt-text-update
branch
from
d1841ff
to
7b5fee0
Compare
…ation, pipeline config page, build now, console output, pipeline syntax, snippet generator. Also associated text are updated.
…tart from stage. Blue ocean image's alt text updated.
…source, branch source configuration, branch indexing. new item image alt text updated and associated text updated.
…ciated text updated.
…nfig global pipeline library, modern scm, legacy scm. Associated text updated and in screenshots branch name changed to main.
… replay sidebar, replay script modified. Associated text updated.
…der icon, organization folder creation, repository resource config, repositories progress builds, organization repositories, PR demo repository branches, stage concurrency, pipeline workflow, fingerprint. Associated text updated and also 'Credentials' line updated.
plugin:docker-workflow[*Docker Pipeline*] plugin:custom-folder-icon[*Custom Folder Icon*] Co-authored-by: Kevin Martens <99040580+kmartens27@users.noreply.github.com>
Co-authored-by: Kevin Martens <99040580+kmartens27@users.noreply.github.com>
ShrushtiDesai
force-pushed
the
pipeline-ss-alt-text-update
branch
from
7b5fee0
to
734f108
Compare
kmartens27
approved these changes
Nov 5, 2024
There was a problem hiding this comment.
thank you very much @ShrushtiDesai for all of the updates and work to get this updated!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR addresses issue #7526.
I have updated the screenshots for the pages under the Pipeline section of the documentation. Additionally, I revised the alt text for the screenshots and made necessary adjustments to the associated text in the documentation where applicable.
I have also incorporated the suggestions from the previous pull request:
Based on your feedback, I’ve committed the text updates and updated the screenshots to align with the naming convention, replacing "master" with "main" where applicable.
On the "Pipeline as Code" page, I also noticed that the Scan Organization process for CloudBeersInc displays three repositories, two of which still reference the "master" branch, as well as the PR demo image where the "master" branch is shown. I have updated these references as well, along with their alt text for accessibility.
I apologize for the inconvenience caused by my mistakes during the force push. Thank you for your patience, and please let me know if further adjustments are needed.