-
Notifications
You must be signed in to change notification settings - Fork 2
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
Improve the app (iOS) #11
Comments
💥 Thanks @lucatorella, we will report all these things to our sponsor and make the app great for next year 👍 |
Thank you for the feedback! @lucatorella Having an entire app ready and perfected for a two-day event is always a challenge because of the lack of time for fixes and because of the dynamic nature of the content. (That said, I did just take care of the feedback page issue 😅) While we can't share our entire framework as that would likely put us out of business, we are intrigued by the talk of @orta 👍 and we will be exploring the idea of making certain parts of it open source. We will look into your feedback and hope you will be there to enjoy a more polished version of the app next year! 😘 |
@ast3150 It's really great that you've put up an app for this new event! My improvement suggestion goes in the direction of "accessibility": Please use a larger font size next time (iOS version, don't know how the Android version looked). I'm not yet in the age not being able to read small text, but I know many people who have that problem (also some with other visual handicaps) and they really made me aware that it's a very, very bad design decision to use very small fonts (or light grey text on white background for example). For example in the side menu there is more than enough space between lines to use a larger font. Best practice of course would be to support iOS's dynamic fonts. |
@danschlet We 100% agree, especially after watching the talk about accessibility. We just need to find a way to implement this without compromising the flexibility we currently have with our backend. (At the moment, content font size is controlled via CSS). But this is definitely on our priority list now! |
Okay! I've looked into this a bit in the last hours, and I'm glad to say: The AppBuilders iOS app now – sort of – supports dynamic type! I changed the CSS for the contents which now scale according to system settings. Of course, for the rest of the UI this will take more effort in terms of design, testing, etc. But this is a bit of a quick-win solution that we will also be using in other projects from now on. cc @danschlet 😃 Edit: Formatting and a word |
Remove the hamburger menu. 'nuff said :) |
But don't get me wrong: I think that the app was very good! I used it a lot. Only add a UITabBarController and it'll be perfect :))))) |
The idea of having an app is great, but the app itself can be improved:
Ok, maybe I'm a bit picky when it comes to iOS apps 😁
Maybe the app should be open-sourced so that we can fix it 🤘 #MovingToOSSByDefault @orta
The text was updated successfully, but these errors were encountered: