Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Bug 1047215 - [Gallery/Video]Background Overlays are not translated #22938

Merged
merged 1 commit into from
Aug 25, 2014

Conversation

russnicoletti
Copy link

This PR incorporates the patch from bug 1038984 as those changes to dialogs.js are required for this patch.

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

nocard2-title = لم يتم العثور على بطاقة الذاكرة
nocard3-text = أدخل بطاقة الذاكرة لمشاهدة الفيديو.
nocard3-title = لم يتم العثور على بطاقة الذاكرة
nocard4-text = أدخل بطاقة الذاكرة لمشاهدة الفيديو.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we really need to rename the l10n key?

IIRC, we only do that when the meaning of sentence changed. I cannot make sure with this. If you have some information about this, please also email me. Thanks.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would suggest to rename as:

nocard4-title = ......
nocard4-text = .....

That makes more consistent.

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@@ -737,36 +739,40 @@ function setPosterImage(dom, poster) {
}

function showOverlay(id) {
currentOverlay = id;
LazyLoader.load('shared/style/confirm.css', function() {
console.log('showOverlay id=' + id);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove this line.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to remove all console.log if they are useless.

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

russnicoletti pushed a commit that referenced this pull request Aug 25, 2014
Bug 1047215 - [Gallery/Video]Background Overlays are not translated r=johu
@russnicoletti russnicoletti merged commit 7e400a8 into mozilla-b2g:master Aug 25, 2014
@russnicoletti russnicoletti deleted the bug-1047215 branch September 23, 2015 18:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants