Skip to content

Port to SFML3#729

Open
thypon wants to merge 1 commit intoSecretchronicles:develfrom
thypon:sfml3-port
Open

Port to SFML3#729
thypon wants to merge 1 commit intoSecretchronicles:develfrom
thypon:sfml3-port

Conversation

@thypon
Copy link
Copy Markdown

@thypon thypon commented Mar 28, 2026

  • Migrate sf::VideoMode to use sf::Vector2u for size (width/height merged into .size.x/.size.y)
  • Replace sf::Event::KeyPressed/KeyReleased union access with evt.getIfsf::Event::KeyPressed/KeyReleased()
  • Add sf::Keyboard::Key:: prefix to all keyboard key enum values
  • Replace sf::Style::Fullscreen with sf::State::Fullscreen and add state parameter to window.create()
  • Change sf::Image::create() to sf::Image::resize() with Vector2u parameter
  • Update collision detection and rect operations to use Vector2f variants
  • Update audio system to use new SFML3 streaming and buffer APIs
  • Replace deprecated sf::Mouse::getPosition with sf::Mouse::getGlobalPosition

- Migrate sf::VideoMode to use sf::Vector2u for size (width/height merged into .size.x/.size.y)
- Replace sf::Event::KeyPressed/KeyReleased union access with evt.getIf<sf::Event::KeyPressed/KeyReleased>()
- Add sf::Keyboard::Key:: prefix to all keyboard key enum values
- Replace sf::Style::Fullscreen with sf::State::Fullscreen and add state parameter to window.create()
- Change sf::Image::create() to sf::Image::resize() with Vector2u parameter
- Update collision detection and rect operations to use Vector2f variants
- Update audio system to use new SFML3 streaming and buffer APIs
- Replace deprecated sf::Mouse::getPosition with sf::Mouse::getGlobalPosition
@Quintus
Copy link
Copy Markdown
Member

Quintus commented Mar 29, 2026

Whoa, that’s a large step. I will try to take a look next weekend, but at the first glance it actually looks fine. I have not tried to compile it yet, though.

Thank you so far.

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