-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
TreeViewNode: Fix arrow pos and size #6574
Conversation
The arrow in the file chooser is too big and in a wrong position when used with a mobile. For full details see [here](https://groups.google.com/forum/#!msg/kivy-users/1bSCsywt2xA/o6F4prAhEAAJ)
Thanks for opening your first pull request here! 💖 Please check out our contributing guidelines. |
Current code screenshots message Current code with out the |
@@ -335,7 +335,7 @@ | |||
|
|||
orientation: 'horizontal' | |||
size_hint_y: None | |||
height: '24dp' # '48dp' if dp(1) > 1 else '24dp' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Too narrow on mobile
@@ -235,8 +235,8 @@ | |||
rgba: 1, 1, 1, int(not self.is_leaf) | |||
Rectangle: | |||
source: 'atlas://data/images/defaulttheme/tree_%s' % ('opened' if self.is_open else 'closed') | |||
size: self.height / 2.0, self.height / 2.0 | |||
pos: self.x - dp(20), self.center_y - dp(8) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Off center positioning in mobile
@@ -235,8 +235,8 @@ | |||
rgba: 1, 1, 1, int(not self.is_leaf) | |||
Rectangle: | |||
source: 'atlas://data/images/defaulttheme/tree_%s' % ('opened' if self.is_open else 'closed') | |||
size: self.height / 2.0, self.height / 2.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Size of arrow gets big if the row height increase in mobile (we must use big rows on mobile)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Congrats on merging your first pull request! 🎉🎉🎉 |
Works for me! @noembryo Nice. Thanks |
The arrow in the file chooser is too big and in a wrong position when used with a mobile.
For full details see here.