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

Bug 1078174 - [Contact] Improve UI on contact settings #24885

Merged
merged 1 commit into from Nov 28, 2014

Conversation

pivanov
Copy link

@pivanov pivanov commented Oct 7, 2014

No description provided.

@try-server-hook
Copy link

pivanov Pavel Ivanov (pivanov) started tests. Results

@try-server-hook
Copy link

pivanov Pavel Ivanov (pivanov) started tests. Results

@try-server-hook
Copy link

pivanov Pavel Ivanov (pivanov) started tests. Results

@try-server-hook
Copy link

pivanov Pavel Ivanov (pivanov) started tests. Results

@try-server-hook
Copy link

pivanov Pavel Ivanov (pivanov) started tests. Results

@try-server-hook
Copy link

pivanov Pavel Ivanov (pivanov) started tests. Results

@@ -574,6 +574,9 @@ contacts.Settings = (function() {
span.textContent = msgPart2;
fbTotalsMsg.appendChild(span);
}

console.log(msgPart1);
console.log(msgPart2);
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

@try-server-hook
Copy link

pivanov Pavel Ivanov (pivanov) started tests. Results

@@ -36,7 +36,7 @@ genericProgress = {{current}}/{{total}}
# Facebook import
facebook = Facebook
facebook-import-msg = {[ plural(imported) ]}
facebook-import-msg[zero] = No friends imported (out of {{total}})
facebook-import-msg[zero] = 0/{{total}} friends imported
facebook-import-msg[one] = One friend imported (out of {{total}})
Copy link
Contributor

Choose a reason for hiding this comment

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

you need to modify this string as well

@try-server-hook
Copy link

pivanov Pavel Ivanov (pivanov) started tests. Results

facebook-import-msg[zero] = No friends imported (out of {{total}})
facebook-import-msg[one] = One friend imported (out of {{total}})
facebook-import-msg[zero] = 0/{{total}} friends imported
facebook-import-msg[one] = 0/{{total}} friends imported
Copy link
Contributor

Choose a reason for hiding this comment

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

That string is incorrect. I think we can solve the entire problem by removing the plural form for this string i.e.

facebook-import-msg = {{imported/{{total}} friends imported

@try-server-hook
Copy link

pivanov Pavel Ivanov (pivanov) started tests. Results

@pivanov
Copy link
Author

pivanov commented Nov 20, 2014

@jmcanterafonseca is it OK now? sorry for all iterations on this one

facebook-import-msg[one] = One friend imported (out of {{total}})
facebook-import-msg = {{imported/{{total}} friends imported
facebook-import-msg[zero] = 0/{{total}} friends imported
facebook-import-msg[one] = 0/{{total}} friends imported
Copy link
Contributor

Choose a reason for hiding this comment

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

line 39 and 40 are no longer needed

@try-server-hook
Copy link

pivanov Pavel Ivanov (pivanov) started tests. Results

@try-server-hook
Copy link

pivanov Pavel Ivanov (pivanov) started tests. Results

@try-server-hook
Copy link

pivanov Pavel Ivanov (pivanov) started tests. Results

@try-server-hook
Copy link

pivanov Pavel Ivanov (pivanov) started tests. Results

@try-server-hook
Copy link

pivanov Pavel Ivanov (pivanov) started tests. Results

@try-server-hook
Copy link

pivanov Pavel Ivanov (pivanov) started tests. Results

arcturus added a commit that referenced this pull request Nov 28, 2014
Bug 1078174 - [Contact] Improve UI on contact settings
@arcturus arcturus merged commit d3c3e86 into mozilla-b2g:master Nov 28, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants