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

[4.2] content history alerts #39611

Merged
merged 3 commits into from
Jan 12, 2023
Merged

[4.2] content history alerts #39611

merged 3 commits into from
Jan 12, 2023

Conversation

brianteeman
Copy link
Contributor

Converts from the ugly browser dependent alerts to use the joomla rendermessage

To test

run npm build:js or use a prebuilt package

Create and save an article and then edit it and save again Click on Versions
Click on all versions and then preview
Click on all versions and then restore
Click on one version and then compare

before - ugly alerts

image

after - joomla less ugly

image

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

Converts from the ugly browser dependent alerts to use the joomla rendermessage

## To test
run npm build:js or use a prebuilt package

Create and save an article and then edit it and save again
Click on Versions
Click on all versions and then preview
### before

### after

Click on all versions and then restore
### before

### after

Click on one version and then compare
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.2-dev labels Jan 12, 2023
@viocassel
Copy link
Contributor

I have tested this item ✅ successfully on 124d167


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39611.

1 similar comment
@dgrammatiko
Copy link
Contributor

I have tested this item ✅ successfully on 124d167


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39611.

@Quy
Copy link
Contributor

Quy commented Jan 12, 2023

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39611.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jan 12, 2023
@roland-d roland-d merged commit 4cf6f00 into joomla:4.2-dev Jan 12, 2023
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jan 12, 2023
@roland-d
Copy link
Contributor

Thank you

@roland-d roland-d added this to the Joomla! 4.2.7 milestone Jan 12, 2023
@brianteeman
Copy link
Contributor Author

thanks for the quick tests and merge

@brianteeman brianteeman deleted the hjistroy branch January 12, 2023 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NPM Resource Changed This Pull Request can't be tested by Patchtester
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants