Page MenuHomePhabricator

scroller: modify to verify bar position variables
Needs ReviewPublic

Authored by akanad on Feb 26 2020, 9:44 PM.

Details

Reviewers
bu5hm4n
Summary

in some complex cases, results of drag_value_get could be invalid.
because variables to calculate the values could change.

Assume that below functionalities are executed in sequence.

content_region_show -> content_resized -> bar_read_and_update

because a formula to calculate vx is that vx = px / (minx + maxx),
and maxx = content width - pan width, vx can not be used to adjust pan position.

this patch check variables to verify vx/y values

Diff Detail

Repository
rEFL core/efl
Branch
scroller
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 16158
Build 10879: arc lint + arc unit
akanad created this revision.Feb 26 2020, 9:44 PM

It seems that this patch has no reviewers specified. If you are unsure who can review your patch, please check this wiki page and see if anyone can be added: https://phab.enlightenment.org/w/maintainers_reviewers/

akanad requested review of this revision.Feb 26 2020, 9:44 PM
zmike added subscribers: bu5hm4n, zmike.

I think @bu5hm4n did work on scrollers the most recently...