Skip to content

[4.1] WebAuthn hints#37203

Merged
bembelimen merged 8 commits intojoomla:4.1-devfrom
kitepascal:4.1-dev
Mar 12, 2022
Merged

[4.1] WebAuthn hints#37203
bembelimen merged 8 commits intojoomla:4.1-devfrom
kitepascal:4.1-dev

Conversation

@kitepascal
Copy link
Copy Markdown
Contributor

Pull Request for Issue # .

Summary of Changes

Testing Instructions

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Documentation Changes Required

Making things clearer :)
Making things clearer
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-4.1-dev labels Mar 5, 2022
PLG_SYSTEM_WEBAUTHN_ERR_NOT_DELETED="Could not remove the authenticator"
PLG_SYSTEM_WEBAUTHN_ERR_USER_REMOVED="The user for this authenticator seems to no longer exist on this site."
PLG_SYSTEM_WEBAUTHN_FIELD_DESC="Lets you manage passwordless login methods using the W3C Web Authentication standard. You need a supported browser and authenticator (eg Google Chrome or Firefox with a FIDO2 certified security key)."
PLG_SYSTEM_WEBAUTHN_FIELD_DESC="Lets you manage passwordless login methods using the W3C Web Authentication standard. You need a supported browser and authenticator (eg Google Chrome or Firefox with a FIDO2 certified security key). <br /><br /><strong>Apple:</strong> Touch/Face ID <br /><strong>Windows:</strong> Hello (Fingerprint / Facial Recognition / PIN) <br /><strong>Android:</strong> Biometric screen lock"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please change to just <br> and end the text with a period

Copy link
Copy Markdown
Contributor

@tecpromotion tecpromotion Mar 5, 2022

Choose a reason for hiding this comment

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

Perhaps also replace Apple as the manufacturer with the name of the operating system "MacOS/iOS/watchOS".
Should Linux be named?

Does Chrome OS now have enough market share and need to be mentioned?
A quick search on Google revealed that Chrome OS now has more market share than macOS.

@kitepascal
Copy link
Copy Markdown
Contributor Author

kitepascal commented Mar 5, 2022

Lets wait for @nikosdion s PR and some more tests.

Joomlas language override fields convert < br > to < br /> on save/reload, so I accepted that.

Should we also link @ceford s detail-documentation directly from PLG_SYSTEM_WEBAUTHN_FIELD_DESC ?

@brianteeman
Copy link
Copy Markdown
Contributor

Joomlas language override fields convert < br > to < br /> on save/reload, so I accepted that.

That's a bug which hopefully will get fixed. Please make the requested chanage

Copy link
Copy Markdown
Contributor Author

@kitepascal kitepascal left a comment

Choose a reason for hiding this comment

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

image

@tecpromotion
Copy link
Copy Markdown
Contributor

tecpromotion commented Mar 5, 2022

There are unnecessary spaces before and after br. Please delete.

Furthermore, I see no need for the last sentence and the link here.

@kitepascal
Copy link
Copy Markdown
Contributor Author

kitepascal commented Mar 5, 2022

I got used to add EOL spaces for anti-glued sentences if somehow line breaks get lost (or just take copy/paste w/o line breaks).

Unlikely here, so I will remove them (cannot hurt anyway).

For the "read more" link, last sentence - I think it is good and very, very useful to have them in scope, like "read more" in frontend. Its just following the intro/short form.

The top right help button is not really in scope and that last sentence with link does not hurt anything/anybody - rather the opposite.
The translations (+ possibly link(s)) and also the help button contents have to be updated anyway, if core Strings get improved for usability.

Maybe theres another 3rd party opinion on that - updated PR will follow.

Removed br spaces and adjusted the link href to the language specific doc.
Inserted /Special:MyLanguage/ also into the link anchor. Was unsure about that before.
Comment thread administrator/language/en-GB/plg_system_webauthn.ini Outdated
Comment thread administrator/language/en-GB/plg_system_webauthn.sys.ini Outdated
Co-authored-by: Stefan Wendhausen <stefan.wendhausen@tec-promotion.de>
@kitepascal
Copy link
Copy Markdown
Contributor Author

Thx @tecpromotion - to change only the one string makes sense to save the translators crowd from unnecessary work (y)

Copy link
Copy Markdown
Contributor

@tecpromotion tecpromotion left a comment

Choose a reason for hiding this comment

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

thanks @kitepascal

@tecpromotion
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on a4a2140


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37203.

1 similar comment
@RickR2H
Copy link
Copy Markdown
Member

RickR2H commented Mar 6, 2022

I have tested this item ✅ successfully on a4a2140


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37203.

@RickR2H
Copy link
Copy Markdown
Member

RickR2H commented Mar 6, 2022

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37203.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Mar 6, 2022
@bembelimen bembelimen changed the title WebAuthn hints [4.1] WebAuthn hints Mar 7, 2022
@bembelimen bembelimen merged commit cb07dbd into joomla:4.1-dev Mar 12, 2022
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Mar 12, 2022
@bembelimen
Copy link
Copy Markdown
Contributor

Thx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language Change This is for Translators

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants