Skip to content

Conversation

@srietkerk
Copy link
Contributor

Closes microsoft/pxt-arcade#7229. It might be good to have someone else try to build this locally since the svgs look a little wonky.. I originally got the icons from font awesome and had copilot change the dimensions/resolution which is why it looks so zoomed in. I needed to make the edits to the icons because they were too small originally.

image

Upload target: https://arcade.makecode.com/app/8f84624f977576ef9912f0d082e2e0c2239e7a4d-efc73b318c#editor

@srietkerk srietkerk requested review from a team and Copilot December 19, 2025 00:16
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Copy link
Member

@riknoll riknoll left a comment

Choose a reason for hiding this comment

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

@srietkerk which version of fontawesome did you take these from? if it's the version we already include, you should be able to just change the button's icon css class to reference the fontawesome icon and remove these files from the svgicon directory entirely.

if these aren't present in our version of fontawesome, then you're going to have to change the icons. these are not resized correctly. i suggest opening them in inkscape and making whatever adjustments you need manually

@srietkerk
Copy link
Contributor Author

I think I just pulled from the latest version, but it looks like there is a version of these arrows in the version that we use as well. Is it possible to have a Semantic UI button reference fontawesome icons?

@srietkerk
Copy link
Contributor Author

Nevermind, I think I see what you mean. I'm testing it now.

@srietkerk
Copy link
Contributor Author

Alright, made the change to just use the font awesome font library. Here's what it looks like:
Tutorial/smaller sim view:
image

Editor:
image

@riknoll
Copy link
Member

riknoll commented Jan 6, 2026

@srietkerk is there a reason you're making this icon bigger than the other icons?

@srietkerk
Copy link
Contributor Author

It doesn't have to be, it's kinda just a personal preference. This is without the size being modified for the regular view and I just think it's hard to tell what it is, especially with all the extra stuff that's happening in the editor.

image image

@srietkerk
Copy link
Contributor Author

I think the minimize icon is worse as well:
image

@riknoll
Copy link
Member

riknoll commented Jan 6, 2026

i think we should keep the icon size consistent. if we want to change it, we should change all of them

@srietkerk
Copy link
Contributor Author

@Jaqster
Copy link
Member

Jaqster commented Jan 6, 2026

This is a minor comment, but is there any way to make the arrows point in the same direction as the ones on YouTube? Just thinking that the more we can create familiar icons, the easier it will be for people to recognize what they mean...
image

@riknoll
Copy link
Member

riknoll commented Jan 6, 2026

@srietkerk you can add the fa-flip-horizontal class to flip them horizontally. also, i think the opacity of all the other icons is set to 0.9, so this one should be too

@srietkerk
Copy link
Contributor Author

Alrighty, flipped the arrows and decreased the opacity of the icon.

image image image

@srietkerk srietkerk merged commit a75bfde into master Jan 7, 2026
20 checks passed
@srietkerk srietkerk deleted the srietkerk/fullscreen-arrows branch January 7, 2026 00:28
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.

Full screen button icons

4 participants