Remove redundant "Italic" style

Created on 1 April 2014, over 11 years ago
Updated 15 September 2023, almost 2 years ago

Problem/Motivation

String::placeholder used everywhere in Drupal (it added Italic style to texts). It's poor readability. Sometimes, I feel it's a historic mistake. Developers copying the code style and unintended to use String::placeholder on drupal coding repeatedly...

It's more worse results in non-English, e.g., in Chinese:

Should it always use Italic only?

Example:
admin/config/people/accounts

A: Italic
Leave empty to use the default system e-mail address (kay@example.com).

B: Bold
Leave empty to use the default system e-mail address (kay@example.com).

Proposed resolution

-

Remaining tasks

- Decide when to use Italic style
- Introduce specified CSS rules to different language system ( [lang="zh-hant"] { font-style: normal; font-weight: bold } )

User interface changes

- font style

API changes

-

📌 Task
Status

Needs review

Version

9.5

Component
Markup 

Last updated 19 days ago

No maintainer
Created by

🇭🇰Hong Kong droplet

Live updates comments and jobs are added and updated live.
  • Usability

    Makes Drupal easier to use. Preferred over UX, D7UX, etc.

  • CSS

    It involves the content or handling of Cascading Style Sheets.

Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • First commit to issue fork.
  • Open in Jenkins → Open on Drupal.org →
    Environment: PHP 8.1 & MySQL 5.7
    last update almost 2 years ago
    30,341 pass
  • Status changed to Needs review almost 2 years ago
  • Status changed to Needs work almost 2 years ago
  • 🇺🇸United States smustgrave

    MR should be 11.x

    Also not sure about this one, have you checked anywhere else on the site it's being used?

    Remaining tasks say

    - Decide when to use Italic style
    - Introduce specified CSS rules to different language system ( [lang="zh-hant"] { font-style: normal; font-weight: bold } )

  • 🇮🇳India BhumikaVarshney Delhi

    Hi droplet,
    As smustgrave mentioned can you please suggest where we need to use italics font or do we need to remove for the help text added for the field or we need to do it for language specific so accordingly we can do the changes.
    Thank You!

  • 🇮🇳India gauravvvv Delhi, India
  • Merge request !5965Rebased with 11.x → (Open) created by Unnamed author
  • 🇮🇳India chetansonawane Gujarat

    chetansonawane changed the visibility of the branch 2230689-11 to hidden.

  • 🇮🇪Ireland markconroy

    Can we get the MR for this updated. The current one says it has over 1,500 commits to it. I doubt we did that much work to this issue :)

    Looking at the specific example from the account settings form, the <em> is only around the email address and not the text.

    '#description' => $this->t("The email address to be used as the 'from' address for all account notifications listed below. If <em>'Visitors, but administrator approval is required'</em> is selected above, a notification email will also be sent to this address for any new registrations. Leave empty to use the default system email address <em>(%site-email).</em>", ['%site-email' => $site_config->get('mail')]),

    If this is still an issue, let's get a list of where we need to change it so we can work on it.

Production build 0.71.5 2024