Thanks to visit codestin.com
Credit goes to github.com

Skip to content

overlays: Add Skylander Portal Overlay#18767

Draft
deReeperJosh wants to merge 10 commits into
RPCS3:masterfrom
deReeperJosh:skylander-portal-overlay
Draft

overlays: Add Skylander Portal Overlay#18767
deReeperJosh wants to merge 10 commits into
RPCS3:masterfrom
deReeperJosh:skylander-portal-overlay

Conversation

@deReeperJosh
Copy link
Copy Markdown
Contributor

Initial development work for #17093

Adds in a new Menu to settings in the overlay to control the Skylander Portal, currently works with clearing and loading with a more complex create skylander user interface required (PR marked as draft until this is complete)

This will be my first contribution to the overlays section, so any feedback/reviews welcome.

New Menu item in the overlay appears above Trophies:

Screenshot 2026-05-19 at 23 31 27

I would like to only conditionally display this (so the menu doesn't get crowded) if the game that is being run is one of the Skylanders/Disney Infinity/Lego Dimensions/Kamen Rider Summonride games, if anyone knows how I could achieve this that would be appreciated.

Tried to make the UI as similar to the Qt Dialog, with a list view and 3 possible button interactions to clear, load and create (create in progress)

Screenshot 2026-05-19 at 23 31 36

When Loading, opens up a simple file selector dialog, where only folders and appropriate file types are displayed, and folders can be navigated through:

Screenshot 2026-05-19 at 23 32 12

As there is also a Qt Dialog, I wanted to make sure that both views shared the same Skylander information, so a few shared methods have been moved to the Skylander.h header file.

@deReeperJosh deReeperJosh changed the title RSX: Add Skylander Portal Overlay overlays: Add Skylander Portal Overlay May 19, 2026
Comment thread rpcs3/Emu/Io/Skylander.h Outdated
Comment thread rpcs3/Emu/Io/Skylander.h Outdated
Comment thread rpcs3/Emu/Io/Skylander.cpp Outdated
Comment thread rpcs3/Emu/Io/Skylander.cpp Outdated
Comment thread rpcs3/Emu/Io/Skylander.cpp Outdated
Comment thread rpcs3/Emu/RSX/Overlays/USBDevices/overlay_skylander_dialog.cpp Outdated
Comment thread rpcs3/Emu/RSX/Overlays/USBDevices/overlay_skylander_dialog.cpp Outdated
Comment thread rpcs3/Emu/RSX/Overlays/USBDevices/overlay_skylander_dialog.cpp Outdated
Comment thread rpcs3/Emu/RSX/Overlays/USBDevices/overlay_skylander_dialog.cpp Outdated
Comment thread rpcs3/Emu/RSX/Overlays/HomeMenu/overlay_home_menu_main_menu.cpp Outdated
@kd-11
Copy link
Copy Markdown
Contributor

kd-11 commented May 19, 2026

UI critiques:

  1. Your list entries are lacking vertical padding which makes the 'file' icons touch the row borders. It's quite ugly like that and easy to fix. Honestly you should have just designed the whole thing off-project then ported it to rpcs3's framework after the fact to avoid this.
  2. You can hide the harsh white border between rows.
  3. You can in fact remove items from the main view if they're unused. Notice how entries like the trophies and friends list isn't being shown in the screenshot because they're not relevant.
  4. The "Emulated Skylander Portal" page is very proof-of-concept-ish. I have no idea what it is supposed to look like - please remember you don't need to copy QT 1:1 here - the controller UI is not based on the QT one at all. I recommend designing the page first then trying to port it to rpcs3, otherwise it will look completely out of place. For example, input prompts must appear at the bottom of the page or on a popup sidebar, not inside the rows.

These may seem nitpicky but they need to be fixed now because nobody is going to go back and work on this later.
Do focus on functionality and architecture first though if that is still in progress.

@deReeperJosh
Copy link
Copy Markdown
Contributor Author

Have slightly tidied up the UI based on the above comments, I don't think the UI will change at all, the list view of each "slot" is how it is best to Load/Clear/Create IMO

Padding has been added to the file/folder icons:

Screenshot 2026-05-20 at 12 17 36

Image buttons have been moved to the bottom of the Dialog:

Screenshot 2026-05-20 at 12 00 22

And the Item in the menu will only display if the portal is activated now

Comment thread rpcs3/rpcs3qt/skylander_dialog.h Outdated
Comment thread rpcs3/Emu/RSX/Overlays/USBDevices/overlay_skylander_dialog.h Outdated
Comment thread rpcs3/Emu/Io/Skylander.cpp Outdated
Comment thread rpcs3/Emu/RSX/Overlays/USBDevices/overlay_skylander_dialog.cpp Outdated
Comment thread rpcs3/Emu/Io/Skylander.h
Comment thread rpcs3/Emu/RSX/Overlays/USBDevices/overlay_skylander_dialog.cpp Outdated
Comment thread rpcs3/Emu/RSX/Overlays/USBDevices/overlay_skylander_dialog.cpp
Comment thread rpcs3/Emu/RSX/Overlays/USBDevices/overlay_skylander_dialog.cpp
@arshkumarsingh
Copy link
Copy Markdown
Contributor

Both error paths now clean, Nice work

@arshkumarsingh
Copy link
Copy Markdown
Contributor

Cause of windows build failure as per github CI logs is usb_device.h including libusb before windows.h and libusb's SSIZE_T typedef conflicts with the newer windows sdk. Maybe moving #include <windows.h> above #include <libusb.h> in usb_device.h should fix

@kd-11 kd-11 self-requested a review May 22, 2026 08:26
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.

4 participants