Skip to content

Classic themes have invisible links #2552

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

Closed
Zorrototo opened this issue Mar 18, 2023 · 7 comments · Fixed by #3054
Closed

Classic themes have invisible links #2552

Zorrototo opened this issue Mar 18, 2023 · 7 comments · Fixed by #3054
Labels
good first issue Good for newcomers.

Comments

@Zorrototo
Copy link

Describe the bug

Link with Classic skins are invisible. For example, default skin has colored links in the Alternative Login method popup, but with Zombie Classic for example, I would expect the links to be green color but they are white like the normal text so we don't see them.

Add logs

(11:18:57) INFO:    [Connection]:       Connectivity: check-online
(11:18:57) INFO:    [Connection]:       Pinging external endpoints
(11:18:57) INFO:    [Legendary]:        Running command: /opt/Heroic/resources/app.asar.unpacked/build/bin/linux/legendary --version
(11:18:57) INFO:    [Legendary]:        Legendary location: /opt/Heroic/resources/app.asar.unpacked/build/bin/linux/legendary
(11:18:57) INFO:    [Gog]:              GOGDL location: /opt/Heroic/resources/app.asar.unpacked/build/bin/linux/gogdl
(11:18:57) INFO:    [Gog]:              GOGDL location: /opt/Heroic/resources/app.asar.unpacked/build/bin/linux/gogdl
(11:18:57) WARNING: [Backend]:          Protocol already registered.
(11:18:57) INFO:    [Gog]:              Running command: /opt/Heroic/resources/app.asar.unpacked/build/bin/linux/gogdl --version
(11:18:57) INFO:    [Connection]:       Connectivity: online
(11:18:58) INFO:    [Backend]:          Loading Screen Ready
(11:18:58) INFO:    [Frontend]:         Refreshing Library
(11:18:58) INFO:    [Legendary]:        Refreshing library...
(11:18:58) INFO:    [Legendary]:        Refreshing Epic Games...
(11:18:58) INFO:    [Legendary]:        Game list updated, got 274 games & DLCs
(11:18:58) INFO:    [Backend]:          Frontend Ready
(11:18:58) INFO:    [Backend]:          AreWeAntiCheatYet data downloaded
(11:18:58) INFO:    [Backend]:          Checking for new Heroic Updates
(11:18:58) INFO:    [Backend]:          Checking for current version changelog
(11:18:58) INFO:    [Backend]:          

Heroic Version: 2.6.2 Trafalgar Law
Legendary Version:  0.20.32 Dark Energy (hotfix #6)
GOGdl Version: 0.6

Electron Version: 22.2.0
Chrome Version: 108.0.5359.215
NodeJS Version: 16.17.1

OS: ManjaroLinux KERNEL: 6.1.19-1-MANJARO ARCH: x64
CPU: AMD Ryzen 5 3600 6-Core Processor @3.6 GOVERNOR: schedutil
RAM: Total: 15.55 GiB Available: 1.33 GiB
GRAPHICS: GPU0: NVIDIA GeForce GTX 1060 6GB VRAM: 6070MB DRIVER: 525.89.02 
PROTOCOL: x11

(11:18:58) INFO:    [Legendary]:        Running command: /opt/Heroic/resources/app.asar.unpacked/build/bin/linux/legendary list --third-party
(11:18:58) INFO:    [Legendary]:        Checking for game updates: /opt/Heroic/resources/app.asar.unpacked/build/bin/linux/legendary list --third-party
(11:19:00) INFO:    [Winetricks]:       Downloading Winetricks
(11:19:00) INFO:    [Legendary]:        Found 0 games to update
(11:19:00) INFO:    [Gog]:              Found 0 game(s) to update
(11:19:03) INFO:    [Backend]:          Starting the Download Queue
(11:19:12) INFO:    [Backend]:          Launching The Talos Principle (Bustard)
(11:19:12) INFO:    [Backend]:          Preventing display from sleep
(11:19:12) INFO:    [Backend]:          Checking if wine version exists: Proton - Proton-GE-Proton7-49
(11:19:22) INFO:    [Backend]:          Checking if wine version exists: Proton - Proton-GE-Proton7-49
(11:19:22) WARNING: [Backend]:          You are using Proton, this can lead to some bugs. Please do not open issues with bugs related to games
(11:19:22) INFO:    [Backend]:          Checking if wine version exists: Proton - Proton-GE-Proton7-49
(11:19:22) INFO:    [Backend]:          Checking if wine version exists: Proton - Proton-GE-Proton7-49
(11:19:22) DEBUG:   [Backend]:          Running Wine command: run wineboot --init
(11:19:26) INFO:    [Backend]:          Created/Updated Wineprefix at /home/omano/Games/Heroic/Prefixes/TheTalosPrinciple
(11:19:26) INFO:    [Legendary]:        Running command: /opt/Heroic/resources/app.asar.unpacked/build/bin/linux/legendary launch Bustard --json --offline
(11:19:26) INFO:    [Legendary]:        Launching The Talos Principle: LD_PRELOAD= STEAM_COMPAT_CLIENT_INSTALL_PATH=/home/omano/.steam/steam STEAM_COMPAT_DATA_PATH=/home/omano/Games/Heroic/Prefixes/TheTalosPrinciple PROTON_NO_ESYNC=1 PROTON_NO_FSYNC=1 STEAM_COMPAT_APP_ID=0 SteamAppId=0 SteamGameId=heroic-TheTalosPrinciple PROTON_LOG_DIR=/home/omano /usr/bin/gamemoderun /opt/Heroic/resources/app.asar.unpacked/build/bin/linux/legendary launch Bustard --language en --no-wine --wrapper "'/home/omano/.config/heroic/tools/proton/Proton-GE-Proton7-49/proton' run"
(11:19:27) INFO:    [Backend]:          Stopping Display Power Saver Blocker
(11:19:27) INFO:    [Frontend]:         Refreshing Library
(11:19:27) INFO:    [Legendary]:        Refreshing library...
(11:19:34) INFO:    [Legendary]:        Logging out: /opt/Heroic/resources/app.asar.unpacked/build/bin/linux/legendary auth --delete
(11:19:34) INFO:    [Legendary]:        Running command: /opt/Heroic/resources/app.asar.unpacked/build/bin/linux/legendary cleanup
(11:19:34) INFO:    [Backend]:          Checking for new Heroic Updates
(11:19:34) INFO:    [Backend]:          Checking for current version changelog
(11:19:51) INFO:    [Legendary]:        Logging in: /opt/Heroic/resources/app.asar.unpacked/build/bin/linux/legendary auth --code <redacted>
(11:19:52) INFO:    [Frontend]:         Refreshing Library
(11:19:52) INFO:    [Legendary]:        Refreshing library...
(11:19:52) INFO:    [Legendary]:        Refreshing Epic Games...
(11:19:52) ERROR:   [Legendary]:        Failed to parse MtMassive.json
(11:19:52) ERROR:   [Legendary]:        Failed to parse bobcat.json
(11:19:52) INFO:    [Legendary]:        Game list updated, got 274 games & DLCs
(11:19:52) INFO:    [Frontend]:         Called Login
(11:19:52) INFO:    [Frontend]:         No cache found, getting data from legendary...
(11:19:52) INFO:    [Legendary]:        Refreshing library...
(11:19:52) ERROR:   [Legendary]:        Failed to parse MtMassive.json
(11:19:52) ERROR:   [Legendary]:        Failed to parse bobcat.json
(11:19:52) INFO:    [Legendary]:        Game list updated, got 274 games & DLCs
(11:19:52) INFO:    [Frontend]:         Force Update
(11:19:52) INFO:    [Legendary]:        Running command: /opt/Heroic/resources/app.asar.unpacked/build/bin/linux/legendary list --third-party
(11:19:54) INFO:    [Frontend]:         Refreshing Library
(11:19:54) INFO:    [Legendary]:        Refreshing library...
(11:19:54) INFO:    [Legendary]:        Refreshing Epic Games...
(11:19:54) ERROR:   [Legendary]:        Failed to parse MtMassive.json
(11:19:54) ERROR:   [Legendary]:        Failed to parse bobcat.json
(11:19:54) INFO:    [Legendary]:        Game list updated, got 274 games & DLCs
(11:19:55) INFO:    [Legendary]:        Running command: /opt/Heroic/resources/app.asar.unpacked/build/bin/linux/legendary list --third-party
(11:19:57) INFO:    [Backend]:          Launching The Talos Principle (Bustard)
(11:19:57) INFO:    [Backend]:          Preventing display from sleep
(11:19:57) INFO:    [Backend]:          Checking if wine version exists: Proton - Proton-GE-Proton7-49
(11:19:58) INFO:    [Backend]:          Checking if wine version exists: Proton - Proton-GE-Proton7-49
(11:19:58) WARNING: [Backend]:          You are using Proton, this can lead to some bugs. Please do not open issues with bugs related to games
(11:19:58) INFO:    [Backend]:          Checking if wine version exists: Proton - Proton-GE-Proton7-49
(11:19:58) INFO:    [Backend]:          Checking if wine version exists: Proton - Proton-GE-Proton7-49
(11:19:58) DEBUG:   [Backend]:          Running Wine command: run wineboot --init
(11:20:00) INFO:    [Legendary]:        Running command: /opt/Heroic/resources/app.asar.unpacked/build/bin/linux/legendary launch Bustard --json --offline
(11:20:00) INFO:    [Legendary]:        Launching The Talos Principle: LD_PRELOAD= STEAM_COMPAT_CLIENT_INSTALL_PATH=/home/omano/.steam/steam STEAM_COMPAT_DATA_PATH=/home/omano/Games/Heroic/Prefixes/TheTalosPrinciple PROTON_NO_ESYNC=1 PROTON_NO_FSYNC=1 STEAM_COMPAT_APP_ID=0 SteamAppId=0 SteamGameId=heroic-TheTalosPrinciple PROTON_LOG_DIR=/home/omano /usr/bin/gamemoderun /opt/Heroic/resources/app.asar.unpacked/build/bin/linux/legendary launch Bustard --language en --no-wine --wrapper "'/home/omano/.config/heroic/tools/proton/Proton-GE-Proton7-49/proton' run"
(11:25:36) INFO:    [Frontend]:         Refreshing Library
(11:25:36) INFO:    [Legendary]:        Refreshing library...
(11:25:36) INFO:    [Legendary]:        Refreshing Epic Games...
(11:25:36) INFO:    [Legendary]:        Game list updated, got 274 games & DLCs
(11:25:36) INFO:    [Legendary]:        Running command: /opt/Heroic/resources/app.asar.unpacked/build/bin/linux/legendary list --third-party

Steps to reproduce

  1. Select Zombie Classic theme or other Classic one
  2. Go to Manage Accounts
  3. Click on Alternative Login method
  4. See the link blends in with the text, it is not colored has it probably should

Expected behavior

Links are highlighted with the theme dominant color

Screenshots

No response

Heroic Version

Latest Stable

System Information

  • OS : Manjaro KDE

Additional information

No response

@Zorrototo
Copy link
Author

Default:
Screenshot_20230321_232920

Zombie Classic:
Screenshot_20230321_233542

We can also see there is no dimming of the background when the popup appears.

@arielj arielj added the good first issue Good for newcomers. label Mar 26, 2023
@Zorrototo
Copy link
Author

2.7.0 didn't fix both issues (link color, and the background dimming effect).

@apokolokynthosis
Copy link
Contributor

apokolokynthosis commented May 10, 2023

@arielj @flavioislima

I'd like to try my best fixing this problem. In which files should I look for that kind of UI and themes issues ?

@apokolokynthosis
Copy link
Contributor

Thanks @arielj ! Should I create a new branch for the pull request or can I use the ui/fix_some_elemts branch ?

@arielj
Copy link
Collaborator

arielj commented May 11, 2023

please create a new branch

@Zorrototo
Copy link
Author

Any progress on this issue with themes?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants