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

7.1 Fix UI glitches #2704

Merged
merged 3 commits into from
Jan 25, 2024
Merged

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Jan 24, 2024

What is this pull request for?

  • Hide element editor errors on save attempt
  • Fix page status icon in page edit mode
  • Fix uploader visibility after upload complete

Checklist

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have added tests to cover this change

@tvdeyen tvdeyen added this to the 7.1 milestone Jan 24, 2024
@tvdeyen tvdeyen requested a review from a team January 24, 2024 14:54
@tvdeyen tvdeyen enabled auto-merge January 24, 2024 17:02
We want to show that a page is online in page edit mode.
It would be confusing otherwise.
We reduce the many times we update the visible and
delegate it to the view where it gets hidden, if the upload
is successful.

Fixes a bug where the upload is still visible on the page edit
screen with upload happening in the assign image dialog.
@tvdeyen tvdeyen force-pushed the hide-elemen-errors-on-save branch from 50dbbb3 to cee4f16 Compare January 25, 2024 11:15
@tvdeyen tvdeyen merged commit 1bb8212 into AlchemyCMS:main Jan 25, 2024
34 checks passed
@tvdeyen tvdeyen deleted the hide-elemen-errors-on-save branch January 25, 2024 11:21
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.

2 participants