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

About roundf() in RDVTabBarItem #105

Open
kirinzer opened this issue Mar 23, 2020 · 3 comments
Open

About roundf() in RDVTabBarItem #105

kirinzer opened this issue Mar 23, 2020 · 3 comments

Comments

@kirinzer
Copy link

Description

I found that use roundf() to calculate rect in RDVTabBarItem is not perfect. Image and title have a different center point in the horizontal. I use floor() to replace roundf(). It works fine. Image and title look like on the right position.

So, could you tell me why not you choice floor()? Thx.

Steps to Reproduce

Use iPhone Plus, X and X Max can reproduce it.

@robbdimitrov
Copy link
Owner

round did the job, but floor should be equally good solution.

@kirinzer
Copy link
Author

All right, do you need a pull request to enhance that?

@kirinzer
Copy link
Author

And I had another question. Why do you set 58 for the tabbar height? In my opinion, the default height is 49 without a safe area. So as a result, I create a subclass and rewrite viewDidLayoutSubviews to change the height. I am not sure it's the right way to use it. So do you agree with it?

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

No branches or pull requests

2 participants