?
avatar Kostelano
Kostelano
24 May 2021

Steps to reproduce the issue

Enable two-factor authentication plugins. Go to editing a user profile (for example, your own). On the Two Factor Authentication tab, step subheadings (or information block subheadings) are not highlighted in any way.

For all field values: One time emergency passwords / Google Authenticator / YubiKey.

Screenshot_2

Below you can see how it looks in Joomla 3.

Screenshot_1

avatar Kostelano Kostelano - open - 24 May 2021
avatar joomla-cms-bot joomla-cms-bot - change - 24 May 2021
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 24 May 2021
avatar brianteeman
brianteeman - comment - 24 May 2021

Caused by this pr #33389 which is setting a fieldset on something that isnt a fieldset

avatar Kostelano
Kostelano - comment - 24 May 2021

@YatharthVyas please take a look at this. Perhaps you will quickly understand what's what.

avatar YatharthVyas
YatharthVyas - comment - 25 May 2021

Caused by this pr #33389 which is setting a fieldset on something that isnt a fieldset

I reverted the changes in PR #33389 but it wasn't enough to fix the issue. I think this is happening because each of the steps is enclosed in an individual fieldset where the heading of the step is a legend tag:

<fieldset>
<legend>
<?php echo Text::_('PLG_TWOFACTORAUTH_TOTP_STEP1_HEAD') ?>
</legend>

image

Edit: This is also the case in J3 which I believe is incorrect as the 'Steps' are not forms and hence they must not be in fieldsets.

image

avatar brianteeman
brianteeman - comment - 25 May 2021

I believe is incorrect as the 'Steps' are not forms and hence they must not be in fieldsets.

Correct they are not. A fieldset (and therefore a legend) are only to be used to group input fields

avatar richard67 richard67 - change - 25 May 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-05-25 14:02:21
Closed_By richard67
avatar richard67 richard67 - close - 25 May 2021
avatar richard67
richard67 - comment - 25 May 2021

Closing as having a pull request. Please test #34205 . Thanks in advance.

Add a Comment

Login with GitHub to post a comment