Skip to content

blitbuffer: RGB32 support for rounded corners#2233

Merged
Frenzie merged 1 commit into
koreader:masterfrom
hius07:blitpuffer-rounded-rect
Jan 19, 2026
Merged

blitbuffer: RGB32 support for rounded corners#2233
Frenzie merged 1 commit into
koreader:masterfrom
hius07:blitpuffer-rounded-rect

Conversation

@hius07

@hius07 hius07 commented Dec 7, 2025

Copy link
Copy Markdown
Member

To be used in colored buttons.


This change is Reviewable

@NiLuJe

NiLuJe commented Dec 7, 2025

Copy link
Copy Markdown
Member

I'd obviously prefer that with the C implementation in, too ;).

(At a quick glance, it wouldn't look too gnarly to implement, just a bunch of code duplication).

@hius07

hius07 commented Dec 8, 2025

Copy link
Copy Markdown
Member Author

Looks like it's already there but the color parsing should be fixed:

koreader-base/blitbuffer.c

Lines 3151 to 3152 in b677605

} else if (bb_type == TYPE_BBRGB32) {
const ColorRGB32 color = { .r = c, .g = c, .b = c, .alpha = 0xFF };

Out of my abilities unfortunately.

@NiLuJe

NiLuJe commented Dec 8, 2025

Copy link
Copy Markdown
Member

Yeah, it already supports every target BB, which is why I mentioned it should be fairly easy (basically the same idea as fill_rect vs. fill_rect_RGB32) ;).

I miiiiight have some time during winter break, but who knows really ;p.

@hius07

hius07 commented Dec 8, 2025

Copy link
Copy Markdown
Member Author

Lua version works fine on Kindle, frontend PR is ready.

@Frenzie Frenzie merged commit d13ab99 into koreader:master Jan 19, 2026
3 checks passed
@hius07 hius07 deleted the blitpuffer-rounded-rect branch January 19, 2026 16:26
@NiLuJe NiLuJe self-assigned this Jan 20, 2026
@NiLuJe

NiLuJe commented Jan 20, 2026

Copy link
Copy Markdown
Member

(Just so I don't forget to look into the missing C bits whenever free time becomes something that exists again xD)

benoit-pierre pushed a commit to benoit-pierre/koreader that referenced this pull request Jan 20, 2026
Frenzie pushed a commit to koreader/koreader that referenced this pull request Jan 28, 2026
Fixes #11033.

Depends on additional change in the available C-bindings from koreader/koreader-base#2260

Also includes koreader/koreader-base#2233
0xstillb pushed a commit to 0xstillb/koreader-thai that referenced this pull request May 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants