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

SITES-16562 - [Xwalk] Open Universal Editor from Franklin Sidekick #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jckautzmann
Copy link
Collaborator

@jckautzmann jckautzmann commented Nov 1, 2023

Copy link

aem-code-sync bot commented Nov 1, 2023

Hello, I'm the AEM Code Sync Bot and I will run some test suites that validate the page speed.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-run PSI Checks

Copy link

aem-code-sync bot commented Nov 1, 2023

Page Scores Audits Google
/us/en PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

async function overrideEditButton(sk) {
const oldEditBtn = await getButton(sk, '.edit.plugin');
const newEditBtn = await getButton(sk, '.aemedit.plugin');
oldEditBtn.replaceWith(newEditBtn);
Copy link
Collaborator

@Buuhuu Buuhuu Nov 2, 2023

Choose a reason for hiding this comment

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

why do we need a new button actually? can we get the old button and just replace the click behaviour?

If so we would not need to add any sidekick config.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I tried it but it didn't work because:

  • the Edit button is disabled
  • even if we enable it, the ootb logic is triggered when the button is clicked and it cannot be overridden

Buuhuu pushed a commit that referenced this pull request Jan 31, 2024
be2e07d doc: update README.md
287e35f feat: adopt the plugin api (#2)
9028520 fix: possible css leaking into pill overlay
162aab2 fix: improve anonymization for better gdpr/hippa compliance (#3)
0071dbd fix: audience pill activation
a486023 fix: audience parsing
4789afd feat: limit the sampling rate
7fbef36 feat: limit the sampling rate (#1)
92e2abb fix: campaigns parsing
4467d3e fix: block-level experiments resolution
4438f52 doc: update readme
44a3aa4 fix: support installation in sub-directories
a77433f style: properly inherit text color in the overlay

git-subtree-dir: plugins/experience-decisioning
git-subtree-split: be2e07ddce1d9c8d1622e6221f7a16593d87b811
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.

None yet

2 participants