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

GamePage image is missing logo when available #2466

Closed
arielj opened this issue Feb 18, 2023 · 2 comments
Closed

GamePage image is missing logo when available #2466

arielj opened this issue Feb 18, 2023 · 2 comments
Assignees
Labels
good first issue Good for newcomers.

Comments

@arielj
Copy link
Collaborator

arielj commented Feb 18, 2023

Describe the bug

For games that have a separate asset for the logo, we are not showing the logo in the game page:

image

image

Add logs

Nothing relevant

Steps to reproduce

  1. Open the game page for any game that has a logo asset (rayman legends, alan wake's, batman arkham, faster than light, celeste, and many more)

Expected behavior

The logo should be placed over the image like it's done in the library

Screenshots

No response

Heroic Version

Latest Stable

System Information

Not relevant

Additional information

No response

@arielj arielj added the good first issue Good for newcomers. label Feb 18, 2023
@thiyagarajan17
Copy link

please assign the issue.. would love to work on it!

@flavioislima
Copy link
Member

please assign the issue.. would love to work on it!

No need to assign. Just let us know that you want to work on that. And also if you are stuck go to our discord for help. 👍

arielj pushed a commit that referenced this issue Nov 25, 2023
* Add logo support in GamePage and GamePicture

In GamePage component, add logo variable and pass it to GamePicture component.
In GamePicture component, add logo parameter; add logo element (same as
the CachedImage logo element in GameCard)

There was no need to add CSS, since there was  already CSS for logo in
GamePicture  that positions it at the bottom center.

* Rename 'logo' variables to 'art_logo'

Purpose of this is to maintain naming consistency with other
variables, such as 'art_cover' or other 'art_...'

* Clean up unnecessary code

* Update logo CSS to look like library's logo
@arielj arielj closed this as completed Nov 25, 2023
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

No branches or pull requests

3 participants