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

fix to: Random Colored Artifacts Showing Up #16

Merged
merged 6 commits into from
Nov 13, 2023
Merged

Conversation

Asamartino
Copy link
Collaborator

Potential solution to issue #8

Copy link
Contributor

@nuke-web3 nuke-web3 left a comment

Choose a reason for hiding this comment

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

Looks like a fix! I removed the commented out html code here, as we do not want this pattern replicated: if we are to use the slides in the book context, and any other, as pure markdown as possible will help avoid any of these kind of issues. Any html we use should be "universal" so it's not dependent on context to work 🤞

@kianenigma I hope you might add some extra speaker notes here for each slide, to describe what is being show stepwise for people without context, and verify this flows as you expected.

@Asamartino where are the source files you created the svgs here with? Perhaps google draw? Or did you modify the svg source itself?

@nuke-web3 nuke-web3 linked an issue Nov 7, 2023 that may be closed by this pull request
@Asamartino
Copy link
Collaborator Author

I modify the .svg source itself using inkscape :)

@Asamartino Asamartino changed the title Fix to random colored artifacts showing up fix to random colored artifacts showing up Nov 8, 2023
@Asamartino Asamartino changed the title fix to random colored artifacts showing up fix to random colored artifacts showing up#8 Nov 8, 2023
@Asamartino Asamartino changed the title fix to random colored artifacts showing up#8 fix Random Colored Artifacts Showing Up #8 Nov 8, 2023
@Asamartino Asamartino changed the title fix Random Colored Artifacts Showing Up #8 fix #8 Nov 8, 2023
@Asamartino Asamartino changed the title fix #8 fix to: Random Colored Artifacts Showing Up #8 Nov 8, 2023
@Asamartino Asamartino changed the title fix to: Random Colored Artifacts Showing Up #8 fix to: Random Colored Artifacts Showing Up Nov 8, 2023
Copy link
Member

@kianenigma kianenigma left a comment

Choose a reason for hiding this comment

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

Okay, bummer that this is a nogo for the book but it seems unavoidable for now.

deleted images to pass the ci-checks
@Asamartino Asamartino merged commit 62ddb88 into main Nov 13, 2023
1 check passed
@Asamartino Asamartino deleted the rectangle_fix branch November 13, 2023 06:31
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.

Random Colored Artifacts Showing Up
3 participants