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.

Sign up for GitHub

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

Jump to bottom

[Focus Mode] New images for the IndexedDB article #2877

Merged
merged 3 commits into from
Oct 19, 2023
Merged

[Focus Mode] New images for the IndexedDB article #2877

merged 3 commits into from
Oct 19, 2023

Conversation

captainbrosset
Copy link
Contributor

@captainbrosset captainbrosset commented Oct 17, 2023 •
edited by mikehoffms
Loading

Rendered article for review:
https://review.learn.microsoft.com/microsoft-edge/devtools-guide-chromium/storage/indexeddb?branch=pr-en-us-2877
Before:
https://learn.microsoft.com/microsoft-edge/devtools-guide-chromium/storage/indexeddb

This PR contains updated images for the IndexedDB article, to match with the Focus Mode UI.

AB#47108543

@captainbrosset captainbrosset added cat: devtools DevTools-related content. focus-mode-screenshots These PRs are about renewing screenshots in the DevTools docs for Focus Mode labels Oct 17, 2023
mikehoffms
mikehoffms requested changes Oct 17, 2023
View reviewed changes
Copy link
Collaborator

@mikehoffms mikehoffms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A Writer/Editor review pass.

@mikehoffms mikehoffms self-requested a review October 19, 2023 00:58
mikehoffms
mikehoffms approved these changes Oct 19, 2023
View reviewed changes
Copy link
Collaborator

@mikehoffms mikehoffms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Writer/Editor 2nd review

@learn-build-service-prod
Copy link

Learn Build status updates of commit 8b56c31:

✅ Validation status: passed

File Status Preview URL Details
microsoft-edge/devtools-guide-chromium/storage/indexeddb.md ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/application-tool-manifest-panel.png ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/clear-icon.png ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/clear-object-store.png ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/clear-site-data.png ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/database-icon.png ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/database-information.png ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/delete-database.png ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/delete-icon.png ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/delete-keyval.png ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/edit-from-snippet.png ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/expanded-indexeddb-menu.png ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/expanded-value.png ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/more-tools-icon.png ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/object-store-icon.png ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/object-store-key-values.png ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/refresh-db.png ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/reload-icon.png ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/select-keyval.png ✅Succeeded View
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/storage-application-clear-storage-indexeddb-clear-site-data.png ✅Succeeded n/a (file deleted or renamed)
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/storage-application-manifest-empty.png ✅Succeeded n/a (file deleted or renamed)
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/storage-application-storage-indexeddb.png ✅Succeeded n/a (file deleted or renamed)
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/storage-application-storage-indexeddb-notes_db.png ✅Succeeded n/a (file deleted or renamed)
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/storage-application-storage-indexeddb-notes_db-delete-database.png ✅Succeeded n/a (file deleted or renamed)
microsoft-edge/devtools-guide-chromium/storage/indexeddb-images/storage-application-storage-indexeddb-notes_db-notes_os.png ✅Succeeded n/a (file deleted or renamed)

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.

For any questions, please:

@captainbrosset captainbrosset merged commit 84ed302 into user/pabrosse/focus-mode-july-2023 Oct 19, 2023
1 check passed
@captainbrosset captainbrosset deleted the user/pabrosse/focus-mode-indexeddb branch October 19, 2023 07:33
@captainbrosset captainbrosset mentioned this pull request Oct 19, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mikehoffms mikehoffms mikehoffms approved these changes

Assignees
No one assigned
Labels
cat: devtools DevTools-related content. focus-mode-screenshots These PRs are about renewing screenshots in the DevTools docs for Focus Mode
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@captainbrosset @mikehoffms