Initial feedback

[jackiew1]

@geek1011, Very nice. I'll use this new toy 😃 A few discussion points in no particular order:

• depending on the ability of the target audience perhaps a default config file (/mnt/onboard/.adds/nm/mystuff) could be included in the KoboRoot.tgz installation file. Non-techie users get a bit panicky if nothing obvious happens after a new install. The one you suggested I try from the other thread should be harmless, or, if screenshots is a bit niche for the average user, maybe just the last line for invert?

  menu_item:main:Screenshots:nickel_setting:screenshots
  menu_item:reader:Screenshots:nickel_setting:screenshots
  menu_item:reader:Invert Screen:nickel_setting:invert

• I think I'll stop using the InvertScreen option in kobopatch in favour of this NickelMenu version. The kobopatch version has that minor niggle of the Reader settings checkbox being in the wrong state after powering off whilst inverted. The NickelMenu version seems to survive reboot OK and it also seems to play nicely with my MiniClock hack when toggling the invert state.

• There is one minor intermittent gremlin I'll mention. Sometimes when toggling the invert state the 'Invert Screen' menu item itself remains on screen. It stands out like a sore thumb because it's either a white box on a mainly black screen or a black box on a mainly white screen. It doesn't particularly bother me because it's removed when you have to refresh the screen to remove the scrubber etc.

[NiLuJe]

I would tend to agree with your assessment about a default config, if only one filled with commented out actual examples (something which, granted, the doc already does, but that fragment could easily be moved on its own to make it less daunting) ;).

(Do note that it does do something by default if no configs are found: it adds a NickelMenu entry that asks the user to read the doc).

The last point is a known issue:

https://github.com/geek1011/NickelMenu/blob/56af7743662263882cad2d70f762cdeb98ec1dfa/src/action_cc.cc#L111

[jackiew1]

(Do note that it does do something by default: it adds a NickelMenu entry that asks the user to read the doc).

Yes, I did notice that but users seem to be more relaxed about editing an existing file than creating a new one from scratch 😃

The last point is a known issue:

Ah, OK, I hadn't realised that.

[jackiew1]

Just an idea for another default option... I don't know if this kind of thing is possible but having a menu item to show a pop-up display of current battery level whilst inside a book may be useful.

I use MiniClock for this myself but most users probably don't. The little battery icon can be visually misleading if you're in NightMode.

[NiLuJe]

One potential quirk of shipping a default config inside the KoboRoot: it'll be wiped by a reinstall.

The other option is creating it programatically only if it doesn't already exist, which is more annoying to implement ;).