Skip to content

feat: preview/publish single + bulk on browse#956

Open
sharanyavinod wants to merge 36 commits into
mainfrom
ew-pub
Open

feat: preview/publish single + bulk on browse#956
sharanyavinod wants to merge 36 commits into
mainfrom
ew-pub

Conversation

@sharanyavinod

@sharanyavinod sharanyavinod commented May 21, 2026

Copy link
Copy Markdown
Contributor

Description

  • Introduces Preview/Publish in the action bar on single and multiple files selection
  • Follows same pattern as delete for the confirm dialog for initial confirm -> switches then to load within action bar + clickable toast to show the urls if need, which auto closes. UX is rudimentary for now, to be revisited when implementing the new designs.
  • For publish, if a subset of the selections have a publish scheduled, user is asked to confirm overriding - all or nothing atm, can iterate if needed to give more granular control
  • Extracts reusable logic from the actions within the edit view

Preview URL: https://ew-pub--da-live--adobe.aem.live/#/exp-workspace/frescopa

Related Issue

Motivation and Context

  • Users can preview directly from browse without stepping into edit and can preview multiple files easily

How Has This Been Tested?

  • Introduced unit tests consistent with existing tests

Screenshots (if appropriate):

Screenshot 2026-05-21 at 16 08 32

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@aem-code-sync

aem-code-sync Bot commented May 21, 2026

Copy link
Copy Markdown

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-sync branch
Commits

@sharanyavinod sharanyavinod changed the title feat: preview single + bulk on browse feat: preview/publish single + bulk on browse May 21, 2026
@sharanyavinod sharanyavinod marked this pull request as draft May 21, 2026 14:29
@sharanyavinod

Copy link
Copy Markdown
Contributor Author

@mhaack Thanks, fixed both of them.

@chrischrischris chrischrischris left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice! I do wonder if we can move some of the code out of da-list into a file we dynamically import when preview/publish is started.

@sharanyavinod

Copy link
Copy Markdown
Contributor Author

Nice! I do wonder if we can move some of the code out of da-list into a file we dynamically import when preview/publish is started.

Indeed - was thinking of handling this in a follow-up PR since we also need to switch the styling to nx2.

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.

3 participants