Skip to content

refactor(renderer): remove dead pinned-overlay scroll short-circuit#463

Open
phanen wants to merge 2 commits into
sudo-tee:mainfrom
phanen:chore/dead-logic
Open

refactor(renderer): remove dead pinned-overlay scroll short-circuit#463
phanen wants to merge 2 commits into
sudo-tee:mainfrom
phanen:chore/dead-logic

Conversation

@phanen

@phanen phanen commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

This never work since a and nil or b is always b

This seems a defensive programming introduced in a52636b

"only once scroll() on dialog first appear" have been implemented here:

local should_scroll = ctx.render_state:get_part('permission-display-part') == nil

if should_scroll then
scroll(true)
end

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.

1 participant