WordPress / two-factor

Two-Factor Authentication for WordPress.
https://wordpress.org/plugins/two-factor/
GNU General Public License v2.0
720 stars 150 forks source link

Replace table tags with div tags to match Application Passwords styles #622

Open christofervas opened 1 month ago

christofervas commented 1 month ago

Is your enhancement related to a problem? Please describe.

The current design of the "Two-Factor Options" section uses HTML tables, which results in a design mismatch compared to the "Application Passwords" section, which utilizes div tags for its headings. This inconsistency is visually noticeable and affects the overall cohesiveness of the UI, as demonstrated in the attached screenshot where basic styles have been applied.

Proposed Solution

To improve the visual consistency and maintain a uniform design language across the profile settings page, I recommend replacing the "Two-Factor Options" heading with div tags, aligning it with the styling approach used in the "Application Passwords" section.

Designs

Screenshot 2024-08-06 122511

Describe alternatives you've considered

No response

Please confirm that you have searched existing issues in this repository.

Yes

dd32 commented 3 weeks ago

Hi @christofervas,

The image you've posted appears to show an interface that's not entirely core WordPress, are you using WordPress.com hosting by chance?

FWIW though, I do agree in changing this to the Application Passwords table format, as it suits that display type better than it does the existing table structure used by other profile fields on the page:

Screenshot 2024-08-19 at 2 59 31 PM
christofervas commented 3 weeks ago

Hey @dd32 thank you for considering this enchantment. Hope to see it in the next release.

This is not a WordPress.com hosting but we have applied some custom styles in the admin area.