902357 - [Notes] Application doesn't recognize if system language is changed by keeping this app in background

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Notes, defect, P3)

Description

[Leo]

Attachment: Menu view options missing

STR:
  1. Select system language as "English" and open the "Notes" application.
  2. Click "Home" button to keep Notes app in background and change the system 
     language to "pt-BR".
  3. Now check the Notes application running in Background.

Expected:
  The locale change should be applied.

Actual:
  Locale change is not applied and options disappeared from Menu view.

Comment 1

[Leo]

Attachment: create notebook

Creating new notebook missing a title

Comment 2

[Leo]

Attachment: notebook title

This is when selecting all notes view.

Comment 3

[Leo]

Attachment: Exit confirm view

Try creating a new note and close the view, the confirm page will be like this.

Comment 4

[Leo]

Attachment: default screen error

Default screen saying "No notes recorded start writing now" is not shown.

Comment 5

[Ran Ben Aharon [:ranbena] EverythingMe]

The app is not authorized to listen to settings changes therefore it cannot react to language change.

If I'm not mistaken, this has been discussed and approved by Dees (which coordinates this project). Never the less, after language change, reopening the app presents the currently selected language.

Comment 6

[Leo]

According to you, if we change the language by keeping this app in background will not reflect on the application immediately but will be applied after restarting the app. This is fine.

But in notes application, the above scenario will end up in broken views as shown in the screenshots, which are not acceptable from user perspective. Instead of displaying these broken views to user, the app should construct the view assuming the previous language. A restart of app can further apply the new language.

My intention here is i don't need a translated view immediately on language change but a view which is not broken will be acceptable here. I am fine to restart the app to apply the new language.

Comment 7

[Matias Crivolotti]

This is not happening to us with latest version, can you please confirm this is still an issue and if it is, please give us some more details on how to reproduce, i.e. using nightly/simulator/device? and from where are you installing the app?

Comment 8

[Leo]

I have tested on Device. Tested again with version 1.0.5
STR:
  1. Set device language as "English".
  2. Open MarketPlace and install "Notes" application.
  3. Open "Notes" application and visit all the views and don't create any notes.
  4. Press "Home" and change the language to "pt-BR" in settings.
  5. Open "Notes" application again and click new note creation icon "+" and cancel 
     the view.
  6. Now open the "Menu" view to find the broken UI.

Please let know if these STR are helpful to reproduce the issue. If not please let know ur IRC ID.

Comment 9

[Matias Crivolotti]

i think the issue here is different, there is a javascript error with indexedDB.
we used IDBTransaction.READ_WRITE which is now deprecated, and should be used as a "readwirte" string instead, this is fixed on version 1.0.6 which is pending approval on marketplace, lets wait till its approved and recheck this then.

thanks.

Comment 10

[Leo]

Ok then. I will test with version 1.0.6 and update you.
thanks

Comment 11

[Andrew Williamson [:eviljeff]]

still broken in 1.0.6.  Developer notified.

Comment 12

[Karen Ward [:kward]]

Nick, will you provide guidance/check in with Matias on the best practice for the app to change language based on device settings?

Comment 13

[(not reading bugmail) Nick Desaulniers [:\n]]

See [0] or [1].

[0] https://gist.github.com/nickdesaulniers/6994463
[1] https://gist.github.com/digitarald/5549799

Comment 14

[Elisa Helton]

Retested on the new Notes version 2.1 on 2 devices:
-Keon Geeksphone FFOS v1.2
-Huawei device FFOS v1.1

The new language is now reflected in the app after downloading Notes, opening app, changing the device language, and then reopening the app. Closing bug.