Instructions for patching firmware 4.25.15875

[geek1011]

Originally Posted by Semwize

If suddenly someone used this patch and suffers from its absence ...

Spoiler:

Increase page navigation history:
  - Enabled: no
  - Description: Increases the number of dots marking navigation history on the scrubber.
  - ReplaceInt: {Offset: , Find: 2, Replace: 4}

Nice! This was one of my favorite patches.

I just looked into it a bit more and it seems that Kobo reverted the change which broke this patch in this version (the change was present from 14672-15676).

I'll make it multi-version and add it to the patch zips again. I'll also add an entry to the patch request post.

Edit: I'd give karma, but apparently I need to spread it around first.

Edit 2: Upon a closer look at 15676, it appears it may have just been compiled optimizations mangling the function. I might have found the relevant code it it, but I'll need to test it.

[Semwize]

@geek1011

It was easy.

But I can't change the font color in Series, it seems I'm just looking in the wrong place. You promised to help

[JSWolf]

Originally Posted by Semwize

@geek1011

It was easy.

But I can't change the font color in Series, it seems I'm just looking in the wrong place. You promised to help

As long as your looking as the settings for Series, can you look where it converts the series name to uppercase and patch it to stop case changing? Thanks.

[Semwize]

Jon,
I have a suspicion that you have very long series names and they are constantly shortening, so you want to remove the uppercase ... Is that right?

[icallaci]

Originally Posted by Semwize

Jon,
I have a suspicion that you have very long series names and they are constantly shortening, so you want to remove the uppercase ... Is that right?

Maybe that's Jon's reason but I dislike series names in uppercase too. I find all uppercase difficult to read at a glance.

[DNSB]

Originally Posted by Semwize

Jon,
I have a suspicion that you have very long series names and they are constantly shortening, so you want to remove the uppercase ... Is that right?

The Kobo GUI shows the series in uppercase. I like this since it makes it easy for me to differentiate between a series and a subtitle. See the attached image for an example.

[Semwize]

Originally Posted by DNSB

See the attached image for an example.

I understood perfectly what he wants.

It just seems to me (as to you) that the Series in uppercase looks more correct. If remove uppercase, series will merge with the title of the book.

Can easily write Series in 2 lines, if it's about the length of the inscription.

[geek1011]

Originally Posted by Semwize

But I can't change the font color in Series, it seems I'm just looking in the wrong place. You promised to help

Yep, I'll take a look later today. I've been busy for the last few weeks.

[JSWolf]

Originally Posted by Semwize

Jon,
I have a suspicion that you have very long series names and they are constantly shortening, so you want to remove the uppercase ... Is that right?

The series names are not too long. But it doesn't look good in all uppercase. To me it would look better with proper upper/lowercase. That why I would like this patched. I would also like this to be added to the standard list of patches.

We do have such a patch for the header/footer.

[geek1011]

Originally Posted by Semwize

@geek1011

It was easy.

But I can't change the font color in Series, it seems I'm just looking in the wrong place. You promised to help

Originally Posted by JSWolf

As long as your looking as the settings for Series, can you look where it converts the series name to uppercase and patch it to stop case changing? Thanks.

Untested, but here's what I found from a quick skim:

libnickel: 0x91f3e0 (nop the uppercase transformation and fix the stack stuff around it, or just replace it with something harmless like QString::trimmed) (edit: apparently, they there's more to it than just this)
nickel: BookInfoWidget.qss [0x14B5163+1316] (just add another rule to #series) (edit: no, this won't work, since it uses an ElidedLabel, which has its own color handling)

[Semwize]

delete

[geek1011]

Originally Posted by Semwize

If suddenly someone used this patch and suffers from its absence ...

Spoiler:

Increase page navigation history:
  - Enabled: no
  - Description: Increases the number of dots marking navigation history on the scrubber.
  - ReplaceInt: {Offset: 0x88BB9C, Find: 2, Replace: 4}

Originally Posted by geek1011

I'll make it multi-version and add it to the patch zips again. I'll also add an entry to the patch request post.

https://github.com/pgaskin/kobopatch...a2f1f26d080f0a

Increase page navigation history:
  - Enabled: no
  - Description: Increases the number of dots marking navigation history on the scrubber.
  # Replace the immediate value for the cmp r1, #2 before the bgt to the inlined
  # QVector::removeFirst() (i.e. QVector::erase(start, start+1)):
  - BaseAddress: "BookmarkHistoryMixin::pushBookmark(Bookmark const&)"
  - ReplaceInt: {Offset: 24, Find: 2, Replace: 4} # note: you can set the replacement to whatever you want, but it MUST be > 1 or bad things will happen

[Dirk-71]

On Mac OS 11.x (Big Sur) there seems to be another approval process!? I do not know how to grant the permissions to run the script. Can you help?

Meanwhile I know what to do:

This command has to be executed before running the script:
chmod +x kobopatch.sh bin/*

[Dirk-71]

DELETE

[genoasalami]

Has anyone been able to get multiple dictionaries working on the latest (November 2020) firmware? I have never been able to get this working and if anyone has step-by-step instructions, I'd appreciate it. I am pretty tech literate and comfortable with coding, but for whatever reason this has never worked for me.

Is there a maximum file size?