Hard coded strings in the prefs general panel

RESOLVED FIXED

Status

Minimo
Minimo Front-End
RESOLVED FIXED
12 years ago
11 years ago

People

(Reporter: Cédric Corazza, Assigned: dougt)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

12 years ago
Hi,
There are two hard coded strings in the prefs general pref panel :
Full Screen Enabled
Homebar Enabled
They shouldn't be.
(Reporter)

Comment 1

12 years ago
Created attachment 228452 [details] [diff] [review]
l10n strings patch

This is my first code patch, so it is likely false ..
Attachment #228452 - Flags: review?

Comment 2

12 years ago
Comment on attachment 228452 [details] [diff] [review]
l10n strings patch

the second part in pane-general.xul is tab indented, spaces seems to be the style in that file
(Assignee)

Comment 3

12 years ago
cool.  checked in the patch (without the ws changes)

Checking in chrome/content/preferences/pane-general.xul;
/cvsroot/mozilla/minimo/chrome/content/preferences/pane-general.xul,v  <--  pane-general.xul
new revision: 1.6.2.6; previous revision: 1.6.2.5
done
Checking in locales/en-US/chrome/preferences.dtd;
/cvsroot/mozilla/minimo/locales/en-US/chrome/Attic/preferences.dtd,v  <--  preferences.dtd
new revision: 1.1.2.2; previous revision: 1.1.2.1
done
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED

Comment 4

11 years ago
Comment on attachment 228452 [details] [diff] [review]
l10n strings patch

Thias patch got checked in. Removing review request
Attachment #228452 - Flags: review?
You need to log in before you can comment on or make changes to this bug.