fix: browser compiling after UIGraph and import changes#1046
Merged
Nottinghster merged 96 commits intoopentibiabr:mainfrom Jan 15, 2025
Merged
fix: browser compiling after UIGraph and import changes#1046Nottinghster merged 96 commits intoopentibiabr:mainfrom
Nottinghster merged 96 commits intoopentibiabr:mainfrom
Conversation
No need to reinvent the wheel
Drop g_ioService to avoid doing work during polling and make the thread sleep a bit if no message ready
- Handle touch events and longpress for right click - Check user agent to determine isMobile - Use VirtualKeyboardAPI to show keyboard (iOS not supported https://developer.mozilla.org/en-US/docs/Web/API/VirtualKeyboard_API)
Adapt load() calls to lua 5.1
kokekanon
approved these changes
Jan 15, 2025
|
vllsystems
pushed a commit
to vllsystems/otclient
that referenced
this pull request
Feb 20, 2025
…1046) * browser support * CMake: WASM should be tested and else left for Linux as before * Remove conflicting vcpkg.json files * Use existing base64 encoder * Check for emscripten before includes and declarations * Improve CMakeLists * Fix case-sensitivity issues * Implement bitlib and restore original .lua files bit operations * Support older protocols and improvements * Set correct numpad keys * Intercept all browser keys * Allow pasting and writing symbols with shift * Improve networking No need to reinvent the wheel * Reload page on client exit * Reduce CPU usage and connection improvements Drop g_ioService to avoid doing work during polling and make the thread sleep a bit if no message ready * Disable Emscripten's text decoder emscripten-core/emscripten#18034 * Basic mobile support - Handle touch events and longpress for right click - Check user agent to determine isMobile - Use VirtualKeyboardAPI to show keyboard (iOS not supported https://developer.mozilla.org/en-US/docs/Web/API/VirtualKeyboard_API) * Improve performance Drop manually swapping buffers to reduce CPU overhead * Case insensitive FS and properly return if websocket fails to be created * Fix Vorbis Linking (7cd3c82) * Allow gameWorld port to be set without rebuilding As servers and login servers are not adapted to send correct information when using the web client, it's necessary to override the world port and ip address that is received. Before, the world port was being forced to what was defined in the WEBPORT variable, not matter what port was informed when connect() was called. Now, you can override the port using lua (characterlist.lua) and avoid rebuilding the client. OBS: Port 7172 will still be overriden to 443 by the client. * Remove unused definition * Remove '&' opentibiabr#894 (comment) * Remove swapBuffers call, we no longer use it opentibiabr@3c6bc5a * Properly encode json opentibiabr#894 (comment) * Update src/framework/net/webconnection.cpp Co-authored-by: divinity76 <divinity76@gmail.com> * Update src/framework/net/webconnection.cpp Co-authored-by: divinity76 <divinity76@gmail.com> * Update src/framework/net/webconnection.cpp Co-authored-by: divinity76 <divinity76@gmail.com> * Update src/framework/platform/browserplatform.cpp Co-authored-by: divinity76 <divinity76@gmail.com> * Update src/framework/platform/browserplatform.cpp Co-authored-by: divinity76 <divinity76@gmail.com> * Update src/framework/platform/browserplatform.cpp Co-authored-by: divinity76 <divinity76@gmail.com> * Update src/framework/platform/browserplatform.cpp Co-authored-by: divinity76 <divinity76@gmail.com> * c++17 remove file * Recommended improvements opentibiabr#894 (comment) opentibiabr#894 (comment) opentibiabr#894 (comment) opentibiabr#894 (comment) * Remove uneeded link libs * remove regex * Cleanup shell.html * Make compiling easier - We can now use vcpkg.json with an adapted cmake command (check guide) - Update overlay-ports to the current baseline * Add baseline to overlay-ports vcpkg.json files * Workflows: ignore overlay-ports vcpkg.json * Bot V8 basic fixes Adapt load() calls to lua 5.1 * Disable UIGraph while the browser building issue is not found * Fix broken regex * Fix broken regex * Fix compiling after include changes --------- Co-authored-by: Ivan Clementino <ivancarlos.clementino@gmail.com> Co-authored-by: Renato Machado <mehahx@gmail.com> Co-authored-by: Rodrigo Paixão <god.rodrigo@hotmail.com> Co-authored-by: Luan Luciano <luanluciano@outlook.com> Co-authored-by: divinity76 <divinity76@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Fix compilation for the browser version after latest changes.