Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added swipedirections for compose #27

Merged
merged 20 commits into from
Aug 12, 2023
Merged

Added swipedirections for compose #27

merged 20 commits into from
Aug 12, 2023

Conversation

KitsuneFolk
Copy link
Contributor

No description provided.

…nt, elevationAmount, elevationColor, scrimColor and swipeDirection.

Added an ability to set a custom elevation color in Compose,
Renamed the property elevation to elevationAmount
@massivemadness
Copy link
Owner

Проверил, в целом всё хорошо работает, только пару мест поправил (кодстайл). Нашел один баг, тень неправильно рендерится при SwipeDirection = RTL и BTT (градиент "внутрь" а не наружу).
Если исправишь то можно мерджить

@KitsuneFolk
Copy link
Contributor Author

KitsuneFolk commented Aug 6, 2023

Проверил, в целом всё хорошо работает, только пару мест поправил (кодстайл). Нашел один баг, тень неправильно рендерится при SwipeDirection = RTL и BTT (градиент "внутрь" а не наружу). Если исправишь то можно мерджить

ok

…ientations got the wrong orientation gradient
@KitsuneFolk
Copy link
Contributor Author

Ахахахах, вот это баг, хотя я хз как ты его нашёл, я там ничего бы и не увидел если бы не изменил цвета на красный и зелёный. И кстати, там цвета инверсированны, startColor рисуется в конце, а endColor в начале, это так и задумано? Ведь это было ещё до моего pr

@massivemadness
Copy link
Owner

startColor рисуется в конце, а endColor в начале

Их надо менять местами в зависимости от направления скролла

@KitsuneFolk
Copy link
Contributor Author

startColor рисуется в конце, а endColor в начале

Их надо менять местами в зависимости от направления скролла

Так они ведь инверсированны даже если swipeDirection стоит left to right

@KitsuneFolk
Copy link
Contributor Author

startColor рисуется в конце, а endColor в начале

Их надо менять местами в зависимости от направления скролла

Так они ведь инверсированны даже если swipeDirection стоит left to right

Пофиксил

@massivemadness massivemadness merged commit d4a3997 into massivemadness:develop Aug 12, 2023
1 check passed
@massivemadness
Copy link
Owner

Поправил баг с мерцанием, замерджил 👍

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.

2 participants