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

Insecure Content Warning Visibility Issue with long content #175

Open
1 task done
ankitguptaindia opened this issue Jul 8, 2024 · 1 comment
Open
1 task done
Labels
type:bug Something isn't working.

Comments

@ankitguptaindia
Copy link
Member

ankitguptaindia commented Jul 8, 2024

Describe the bug

Description of the issue:

When a post or page contains long content and triggers an insecure content warning, the page should automatically scroll to the top and highlight the error. Currently, if a post/page with long content has insecure content, after clicking the publish/update button, the user is unable to identify the cause of the failure because the warning appears at the top of the screen, which may not be visible due to the length of the content.

Recording.737.mp4

Steps to Reproduce

Steps to Reproduce:

  1. Create or edit a post/page.
  2. Add a significant amount of content to the editor to make the page scrollable.
  3. Insert insecure content
  4. Click the publish/update button.

Screenshots, screen recording, code snippet

Expected Result:

The page should automatically scroll to the top and highlight the insecure content warning, making it clear to the user why the post/page is not published.

Environment information

No response

WordPress information

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@ankitguptaindia ankitguptaindia added the type:bug Something isn't working. label Jul 8, 2024
@jeffpaul jeffpaul added this to the Future Release milestone Jul 8, 2024
@jeffpaul jeffpaul moved this from Incoming to Backlog in Open Source Practice Jul 8, 2024
@ajmaurya99
Copy link

Tested out the plugin on the latest WP Version [Version 6.6.1]

I can see the plugin as expected and he insecure warning is always displayed above the content, No matter how much long the post is.

Tested on Chrome, Firefox and Safari.

I have tested both the use cases,

  1. Publishing the post for the first time. - https://share.zight.com/6quoqNr1#discussion
  2. Updating an already published post. - https://share.zight.com/d5uXKy6D#discussion

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Something isn't working.
Projects
Status: Backlog
Development

No branches or pull requests

3 participants