Under Review
over 1 year ago

bug in editing settings groups

I uploaded a beta datafield to the store to be able to debug the new settings groups: https://apps.garmin.com/en-US/apps/8cfc5bd9-0835-4522-b3a0-fd3dfbfc58d2

When I edit the settings in Connect IQ Store Android the settings are "lost".

To reproduce:

1. this is my settings.xml:

<settings>
  <group id="mainGroup" title="@Strings.AppName" description="@Strings.settingsIntHrGroupTitle">
    <setting propertyKey="@Properties.hrmAntID" title="@Strings.settingsExtHrmAntId" prompt="@Strings.settingsExtHrmAntIdPrompt">
      <settingConfig type="numeric" min="0"/>
    </setting>
    <setting propertyKey="@Properties.showIntHr" title="@Strings.settingsShowIntHr">
      <settingConfig type="boolean"/>
    </setting>
  </group>
  <group id="intHRGroup" title="@Strings.settingsIntHrGroupTitle" description="@Strings.settingsIntHrGroupDescription" enableIfTrue="@Properties.showIntHr">
    <setting propertyKey="@Properties.intHrSide" title="@Strings.settingsIntHrSide">
      <settingConfig type="boolean"/>
    </setting>
    <setting propertyKey="@Properties.intHrPrefix" title="@Strings.settingsIntHrPrefixTitle">
      <settingConfig type="alphaNumeric"/>
    </setting>
    <setting propertyKey="@Properties.extHrPrefix" title="@Strings.settingsExtHrPrefixTitle">
      <settingConfig type="alphaNumeric"/>
    </setting>
  </group>
</settings>

2. start to edit settings in android

3. enable showIntHr. This will correctly (though very inconveniently*) display intHRGroup as it's dependent on it (enableIfTrue="@Properties.showIntHr")

4. enter the group settings

5. go back to the main settings (either using the left arrow in the top-left corner or by using the android device's back button)

6. you'll see that the previous changes in the settings are lost

*) I would also argue that the way the grouped settings seems to be implemented is not user friendly. I think it would be much better if the group would be displayed in the same page (in the main settings page) instead of opening them in a new page. For example when I see now the group's title, and the ">" in the right side, clicking on it I would expect to see the group settings opened below it (probably changing the right arrow to a down arrow)

See also: https://forums.garmin.com/apps-software/mobile-apps-web/f/connect-iq-store-android/289708/bug-in-editing-settings-groups

  • Next thing is scrolling up after returning from groups.

    The strange thing is that there more than one code for settings - sim properties runs well and probably the it runs exactly same way because when no internet there is the error

    so what is the problem to merge code and fix quickly bug?

  • One more thing. End user can change settings in

    A mobile app - groups are grouping and you can change it in another window including only properties from group

    B. express - looking is almost like sim app settings editor

    Of course I think both A/B can be much better (and without of bugs of course) but in B-

    - there is no in layout end of group

    - labels - fatal - line break, alignment, etc. I can't understand how is it possible make such trivial things are so badly made .Why they simply looks like:

    --------------------------------
    Group name
      prp1
        value
      prp2
        value
    --------------------------------
    prp3
       value

    When we can expect the bugs reading from default will fix? The bugs means that there was no test carried out on the new functionality - only what works is the sim app settings editor.

  • yes I confirm settings in groups are saved but each opening settings load default values not set

  • Same here. I implemented "group" in one app - next day I received a bunch of complaints that settings are not working. Indeed, all settings went to default values when opened in Connect IQ app. PLEASE fix it!

  • A related issue: On the iOS Connect IQ app (Version 2.15), when you leave a settings group, the "Save" button disappears entirely from the screen. Exiting the settings menu does not save any changes.