jruesga / CMFileManager

A file manager for CyanogenMod
26 stars 219 forks source link

File system info box shows wrong warning message #17

Closed DvTonder closed 12 years ago

DvTonder commented 12 years ago

The dialog box shows that "file system mounting operations are not allowed in non-privileged mode' when I am running with privileged (advanced) mode on

DvTonder commented 12 years ago

Looking at it now, I see the ability to select privileged and non-privileged console in the menu - how is that different from advanced mode? are they related?

jruesga commented 12 years ago

3 navigation modes:

The different is that app ask the user when a privileged action is required

DvTonder commented 12 years ago

Hmm, I wonder then if it would not be better to have those as part of the settings itself - just like you set the long-press action, maybe it would be better from a UX perspective to change the current 'Privileged mode' settings item (was Advanced mode until I changed it yesterday) from a checkbox to a single select list item called 'Access mode' (or something similar) with 3 options - 'Safe mode' (default), 'Prompt user mode' and 'Root access mode'. That way the item is set in one place.

It would be ideal to then change the strings references to 'console' to something that is more user friendly for non-technical users.

jruesga commented 12 years ago

Yes, you are right. More to the list.

jruesga commented 12 years ago

Done.