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

[Renderers/Allegro4] Add Allegro 4.x renderer and demo #267

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

eigenlenk
Copy link

@eigenlenk eigenlenk commented Feb 16, 2025

Allegro 4 renderer

API conformance

  • Rectangle (+rounded)
  • Text
  • Border (+rounded)
  • Scissor
  • Custom (accepts a function pointer to allow custom drawing within element)

Missing or incomplete features

  • Image (missing resizing support and corner rounding)

Allegro 4 demo

Demo is rather random/silly and based on this screenshot of Microsoft Encarta 95:
image

Remade in Clay:
image

Same demo compiled for MS-DOS:
image


So yeah, perhaps not the most useful renderer and demo since 4.x is rather old. But now that I have a better grasp of the library, I could try supporting the more modern 5.x version which is still actively developed. I also couldn't work out fetching and compiling the library through Make, so a pre-built binary is included instead 🤷‍♂️

@eigenlenk eigenlenk force-pushed the feature/add-allegro4-renderer branch from 56c0397 to 9fe588a Compare February 17, 2025 08:28
@nicbarker
Copy link
Owner

Hello, sorry for the delay in getting to this. It looks so awesome 😁
Don't worry about not including the .dll, I'm sure people who are motivated to use the Allegro renderer will be able to source a copy of the lib themselves 🙂
Thank you for the hard work!

@eigenlenk
Copy link
Author

I think I need to drop Make support for this and include a README instead. It's not going to build on Linux out of the box anyway, and only works with MinGW on Windows as well. Should silence these failing checks.

@eigenlenk eigenlenk force-pushed the feature/add-allegro4-renderer branch from 9fe588a to e5cf195 Compare February 20, 2025 09:47
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.

3 participants