Kindle -- KOReader: a document reader for PDF, DJVU, EPUB, FB2, HTML, ... (GPLv3)

[ilovejedd]

Still unresponsive.

Tried to be a bit methodical in the touch input if that helps any.

1. Tap 4 corners: top left, top right, bottom left, bottom right (may have mixed up the bottom ones)
   
2. Tap center edges (cross pattern): top center, bottom center, left center, right center.
   
3. Horizontal swipes (l to r): top, center, bottom
   
4. Vertical swipes (t to b): left, center, right
   
5. Diagonal swipes: top left - bottom right (& vice versa), top right - bottom left (& vice versa)

I may have switched #4 and #5. Ugh, I'm too young for my memory to be failing.

[makara]

I want to change the size of the upper status bar, for this I insert a new parameter in settings.reader.lua:
["cre_header_status_font_size"] = 40, but this entry is not saved.
I edit using TextEdit.app on MacOS.

[Frenzie]

What do you mean by it isn't saved? Are you doing this while KOReader is running?

[jhh]

Originally Posted by NiLuJe

@jhh: Are you also still on the factory FW?

Now my KT4 in on firmware 5.11.2. Still no luck with KOReader nightly, it's unresponsive.
If you need shell access, I haven't managed to set it up yet (I get APIPA IP).

[makara]

Frenzie
I save the document, reload KOREader and nothing changes in the interface. Again I open settings.reader.lua and no changes have been saved to it.

[Frenzie]

If KOReader overwrites your changes it means it's restoring them from the backup, i.e., you introduced a syntax error. Which is weird, because your fragment works for me.

It needs to go in between the return {}:

return {
  ["cre_header_status_font_size"] = 40,
}

Unless the comma was part of your sentence and not part of what you actually wrote. The comma is an important part of the syntax.

[makara]

The comma is part of the text. Thank. I'll try again, but more carefully.

[makara]

I do not understand the location of { } at the beginning and at the end.

-- we can read Lua syntax here!
return {
["cre_header_status_font_size"] = 40,
["copt_line_spacing"] = 110,
["disable_double_tap"] = true,
["copt_render_dpi"] = 300,
["start_with"] = "last",
["disable_kobolight"] = true,
["cre_header_status_font_size"] = 30,
["full_refresh_count"] = 12,
["larger_tap_area_to_follow_links"] = true,
["copt_b_page_margin"] = 10,
["gesture_fm"] = {

that doesn't work.

[Frenzie]

Originally Posted by makara

that doesn't work.

It should, unless that represents the actual end result instead of merely the first few lines. A complete file would look like this:

return {
    ["cre_header_status_font_size"] = 40,
    ["copt_line_spacing"] = 110,
    ["disable_double_tap"] = true,
    ["copt_render_dpi"] = 300,
    ["start_with"] = "last",
    ["disable_kobolight"] = true,
    ["cre_header_status_font_size"] = 30,
    ["full_refresh_count"] = 12,
    ["larger_tap_area_to_follow_links"] = true,
    ["copt_b_page_margin"] = 10,
}

Any opening and closing brackets have to match.

[makara]

Nothing happens. I'll go tomorrow. Thank!

[shamanNS]

Last "element" in that "options list" shouldn't end with a comma. That is a syntax error.

[Frenzie]

Originally Posted by shamanNS

Last "element" in that "options list" shouldn't end with a comma. That is a syntax error.

Not only is it not a syntax error, but removing trailing commas is bad practice.

[NiLuJe]

@jhh: Thanks for double-checking .

Unfortunately, input handling gives me hives, so, this is going to need a willing dev with the device to figure out :/.

It's *probably* something stupidly simple, but, still, hard (and/or annoying ;p) to remotely debug, even if I actually liked dealing with input crap ^^.

EDIT: Opened #5070 to track it.

[makara]

Originally Posted by shamanNS

Last "element" in that "options list" shouldn't end with a comma. That is a syntax error.

This is not my fault. On my two books, Kindle and Aura are the same.

[jhh]

Ok, @NiLuJe and @ilovejedd, thanks a lot for your work so far!