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

Use code to draw button effect instead of pictures #718

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

howard0su
Copy link
Contributor

@howard0su howard0su commented Feb 19, 2019

This saves SPI flash space for the color screen Tx with small SPI flash or no flash.

@PhracturedBlue
Copy link
Contributor

Does this have real ROI and actually make widgets easier to use? It will require users to update their filesystem...something we try not to require as much as possible. I tested that it doesn't crash when the old images are used, but it also will appear to be a graphics bug in that case.

{NULL, 64, 16, 0, 0}, /*FILE_BTN64_16 */
{NULL, 48, 16, 0, 0}, /*FILE_BTN48_16 */
{NULL, 32, 16, 0, 0}, /*FILE_BTN32_16 */
{NULL,192, 16, 1, 0}, /*FILE_SPIN192 */
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lint Error:

41:  Missing space after ,  

u16 uppercolor = 0;
u16 bottomcolor = 0;

switch(map->x_off) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lint Error:

819:  Missing space before ( in switch(  

y_end = size / 3;

// draw additional arrows
switch(map->y_off)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lint Error:

879:  Missing space before ( in switch(  

@howard0su howard0su changed the title combine spin normal/press image Use code to draw button effect instead of pictures Mar 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants