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

Upstream 6b774f2dfad7f8097b80539db30cfa27a355d299 #9383

Merged
merged 3 commits into from
Jul 12, 2023

Conversation

pascalwengerter
Copy link
Contributor


return {
title: $gettext('Blocked pop-ups and redirects'),
timeout: 20,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@elizavetaRa 10s felt too short, so I doubled it. Still quite a lot of text to digest for the average user though

@pascalwengerter
Copy link
Contributor Author

Random CI failures..

Copy link
Member

@dschmidt dschmidt left a comment

Choose a reason for hiding this comment

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

We were somehow not talking about the same thing... I meant, we should put the whole window open and message code into the composable - not just the message...

const { $gettext } = useGettext()

return {
title: $gettext('Blocked pop-ups and redirects'),
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use a slightly different title to make clear, that blocking is not a feature but that the app detected, that pop ups are getting blocked:
"Pop-up and redirect block detected"

timeout: 20,
status: 'warning',
desc: $gettext(
'Some features might not work correctly. Please enable pop-ups and redirects in Settings > Privacy & Security > Site Settings > Permissions'
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use a more generic message that applies to all browsers:
"Please turn on pop-ups and redirects in your browser settings to make sure everything works right."

@pascalwengerter pascalwengerter requested a review from dschmidt July 12, 2023 06:42
@dschmidt dschmidt enabled auto-merge (squash) July 12, 2023 06:53
@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

33.3% 33.3% Coverage
0.0% 0.0% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@dschmidt dschmidt merged commit e83c42a into owncloud:master Jul 12, 2023
@pascalwengerter pascalwengerter deleted the feature/9377 branch July 13, 2023 16:14
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.

Warning of disabled popups by window.open actions
3 participants