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

2023.9.1: Add Bitmap Stack screen #140

Merged
merged 31 commits into from
Nov 10, 2023

Conversation

andrewjswan
Copy link

@lubeda
Copy link
Owner

lubeda commented Nov 9, 2023

What about the readme? Why did you change calc scroll time?

@andrewjswan
Copy link
Author

I dont change, only add new for icons

    void calc_scroll_time(std::string text, uint16_t screen_time);
    void calc_scroll_time(uint8_t icon_count, uint16_t screen_time);

Readme updated ...

@lubeda lubeda merged commit c3350db into lubeda:2023.9.1 Nov 10, 2023
5 checks passed
@andrewjswan andrewjswan deleted the 2023.9.1-Add_Bitmap_stack_screen branch November 10, 2023 16:00
@lubeda
Copy link
Owner

lubeda commented Nov 10, 2023

Hi, its a nice effect but the |two "parameter" isn't a clean way to pass parameters and the functionality isn't described in the readme.

@andrewjswan
Copy link
Author

I added 2 examples at the bottom of the Readme, I just don't quite understand how to describe it clearly, should see the effect working on the 1,2,3 icon.

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