Closed Bug 1878651 Opened 1 year ago Closed 1 year ago

Tooltip suggestion translates the css value

Categories

(DevTools :: Inspector: Rules, defect, P3)

Firefox 122
defect

Tracking

(firefox124 fixed)

RESOLVED FIXED
124 Branch
Tracking Status
firefox124 --- fixed

People

(Reporter: jeanphilippe.green, Assigned: jdescottes)

Details

Attachments

(2 files)

Attached image firefox-bug.png

Steps to reproduce:

With Swedish language package enabled, in the developer tools' rules editor:

Add the following rule on an element that doesn't have display inline or table-cell :
vertical-align: middle;

An info-icon (circled "i") appears, hover over it.

Actual results:

The suggestion tells me to add display:tabell-cell, which seems to be a translated version of display: table-cell. Notice the word "tabell" instead of "table"

Expected results:

It should say display:table-cell, since css doesn't have a value tabell-cell for display.

The Bugbug bot thinks this bug should belong to the 'DevTools::Inspector: Rules' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Component: Untriaged → Inspector: Rules
Product: Firefox → DevTools

We don't indicate that the property name and values shouldn't be translated in https://searchfox.org/mozilla-central/rev/26dd94a024f21971e539a8650c2f177323c98fe2/devtools/client/locales/en-US/tooltips.ftl#78-80 , so localizers might try to translate those :/

Hi Francesco,

What is the process to update a localized string for a specific locale (here only the swedish version seems wrong)? Should we file a bug?
On our side we will update the note to make it clear that all the properties and values in this section should not be localized.

Severity: -- → S3
Status: UNCONFIRMED → NEW
Type: enhancement → defect
Ever confirmed: true
Flags: needinfo?(francesco.lodolo)
Priority: -- → P3

File a bug in Mozilla Localizations::(locale)

In this case, I just fixed it directly
https://pontoon.mozilla.org/sv-SE/firefox/all-resources/?search=tabell-cell&string=199620

Flags: needinfo?(francesco.lodolo)
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/97a25fa6638e [devtools] Update localization note for inactive CSS tooltips r=devtools-reviewers,fluent-reviewers,nchevobbe,flod
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 124 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: