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

Site navigation links resize their icon to 800px when using uploaded images. #3712

Closed
HatchlingByHeart opened this issue Mar 5, 2019 · 3 comments
Labels
type: bug A problem that should not be happening
Milestone

Comments

@HatchlingByHeart
Copy link

HatchlingByHeart commented Mar 5, 2019

As title says.

Icons work as intended (they don't get resized) if they came pre-installed with e107 or belong to a plugin. The issue only affects images uploaded via media manager.

The issue was not solved by using another theme or removing all plugins, so I can only guess it is not theme or plugin related.

Example Image: https://www.dropbox.com/s/44bqam45l2qgdg0/ss_e107.png?dl=0
(News is using an uploaded 64x64 png file, the rest of the links are using the icon from the ReCaptcha plugin)

e107 Version: 2.1.9
OS: Debian 4.9.130-2
Apache Version: Apache/2.4.25
PHP Version: 7.1.26-2+0-20190216175340.13+stretch-1.gbpc20626
MySQL (MariaDB) Version: 10.1.37-MariaDB-0+deb9u1

@Moc Moc added the type: bug A problem that should not be happening label Mar 6, 2019
@CaMer0n CaMer0n added this to the e107 2.2.0 milestone Mar 6, 2019
@CaMer0n CaMer0n closed this as completed in d05d4f6 Mar 6, 2019
@CaMer0n
Copy link
Member

CaMer0n commented Mar 6, 2019

Thank you @HatchlingByHeart If you update from github and then re-upload the icon, it should now be placed in the correct folder (icons) and render correctly.

@HatchlingByHeart
Copy link
Author

Thank you, @CaMer0n. It works perfectly now after replacing e_parse_class.php and media_class.php with latest versions from github.

@CaMer0n
Copy link
Member

CaMer0n commented Apr 6, 2019

@HatchlingByHeart Great! Thank you for the feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A problem that should not be happening
Projects
None yet
Development

No branches or pull requests

3 participants