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.

Sign up for GitHub

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

Jump to bottom

[QA] - Develop branch #515

Closed
5 tasks done
pavanpatil1 opened this issue Feb 20, 2024 · 7 comments
Closed
5 tasks done

[QA] - Develop branch #515

pavanpatil1 opened this issue Feb 20, 2024 · 7 comments

Comments

@pavanpatil1
Copy link
Collaborator

pavanpatil1 commented Feb 20, 2024 •
edited by mohdsayed
Loading

@pavanpatil1
Copy link
Collaborator Author

  • The background color of the allowed domain is not persist. It not visible for sometime.

Steps to reproduce:

  1. Enable the CDP and from cookie frame allow any domain.
  2. Go to another frame and come back to frame where allowed the domain.
  3. Observe the color.
allowlistissue.webm

@pavanpatil1
Copy link
Collaborator Author

  • There should not be extra padding before the cookie blocking response message.

image

@pavanpatil1
Copy link
Collaborator Author

  • In the CLI, In the affected cookies section - The blocked reason for the cookies is not visible.

image

@pavanpatil1
Copy link
Collaborator Author

pavanpatil1 commented Feb 20, 2024 •
edited
Loading

  • In CLI, In the unknown frames - The newly added icon is visible however cookie is not blocked. It should not be visible in CLI

Note: Observed this on all the cookies which are not blocked.

image

@pavanpatil1
Copy link
Collaborator Author

pavanpatil1 commented Feb 20, 2024 •
edited
Loading

  • In CLI, Technology frame values are visible in yellow background. It should not show the yellow color in the technology frame As it visible only when cookies are blocked.

image

@mohdsayed
Copy link
Collaborator

The issue was fixed in #518

@pavanpatil1
Copy link
Collaborator Author

All issue are fixed and merged. Hence closing this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mohdsayed @pavanpatil1