-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
On the login screen a warning will pop up if an unsupported/outdated browser is used #26967
Conversation
@DeepDiver1975, thanks for your PR! By analyzing the history of the files in this pull request, we identified @bartv2, @vincchan and @butonic to be potential reviewers. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great idea. Maybe not annoying enough 😉
Please remove the commented out line then it's good to go 👍
@@ -1,10 +1,12 @@ | |||
<?php /** @var $l \OCP\IL10N */ ?> | |||
<?php | |||
vendor_script('jsTimezoneDetect/jstz'); | |||
//vendor_script('browser-update/browser-update'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove?
@@ -91,4 +93,4 @@ | |||
</ul> | |||
</fieldset> | |||
</form> | |||
<?php } | |||
<?php } ?> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, I thought we usually never close the php block at the end of files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fine 👍 Maybe change the text like this if possible (if not, fine as well): "Aktualisieren Sie Ihren Browser..."
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Description
We should notify the user if an outdated browser is used
Motivation and Context
Screenshots (if appropriate):
Types of changes
Checklist: