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

Mapbox license not visible #138

Closed
illabo opened this issue Aug 6, 2020 · 7 comments · Fixed by #144
Closed

Mapbox license not visible #138

illabo opened this issue Aug 6, 2020 · 7 comments · Fixed by #144

Comments

@illabo
Copy link

illabo commented Aug 6, 2020

Hello there!

On older iOS versions (tried on iOS 10.3 and 12.1) Mapbox-iOS-SDK license not visible in settings. However scroll view scrolls vertically a lot as if the text is visible actually. When copy paste it to txt file and load it manually from file set in license_plist.yml it is invisible too. In iOS 13 it is displayed ok if not crashing settings app.

Perhaps there are some illegal characters in the text? Maybe texts should be preparsed escaping such a characters?

@illabo
Copy link
Author

illabo commented Aug 11, 2020

I've tried running my app on even older device. On iPhone 4s with iOS 9.3.6 all the licenses are offset one screen height from top. However Mapbox-iOS-SDK license is visible! Thinking I was wrong not to scroll it on iOS 10—12 all way down, maybe it's just offset too. But no.

All the licenses are visible ok, except Mapbox's. And it isn't offset as on iOS 9.

@Cyberbeni
Copy link
Contributor

Maybe it is too long. What happens if you split it into smaller chunks, can all of them render?

@illabo
Copy link
Author

illabo commented Oct 10, 2020

Thanks for suggestion. Maybe its length is the reason however other quite long texts are visible ok.
Haven’t tried splitting as it is against of idea of licenses convenient presentation.

@Cyberbeni
Copy link
Contributor

I mean we are jamming all the text into one footer, which probably uses a UILabel, which doesn't care about not showing all its content inside the tableview's scrollview, so it tries to render all the text at once. We could try splitting up the paragraphs and use separate elements for them if that is the cause of the problem.

@Cyberbeni
Copy link
Contributor

Cyberbeni commented Oct 15, 2020

@illabo could you try my version from here? https://github.com/Cyberbeni/LicensePlist/tree/multiple-footers

Installing from source: https://github.com/mono0926/LicensePlist#from-source

It splits double newlines to different footers, which results in a slightly bigger gap between them but if it solves the issue then I think that's acceptable.

I initially also converted single newlines to spaces but then I realised maybe not all licences in markdown, so I'll leave that for another issue.

@Cyberbeni
Copy link
Contributor

It works on iOS 10.3 simulator
The gaps are way bigger on iOS14, might be better to split on horizontal lines.

@illabo
Copy link
Author

illabo commented Oct 17, 2020

@Cyberbeni Sorry, I haven’t checked your fork yet. The issue is only present in older iOS versions. Starting with iOS 13 everything looks great. Maybe it’s better to check iOS version before splitting the long text?
Sorry again, I definitely have to take a look at your version.

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 a pull request may close this issue.

2 participants