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

[fix] Red squares instead of gif images #101

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Dzordzu
Copy link

@Dzordzu Dzordzu commented Jan 2, 2025

Issue

Red squares are displayed in places where gif symbols were present.

Changes

  • Changed gif to png in all consts
  • Converted gif files to png
  • Updated xmls to include pngs

Image

Left side - issue
Right side - resolved (this PR)

image

Important notes

After the startup and before clicking 'LDAP Servers' and 'Search Logs' are still in red squares. After clicking the problem disappears.

image

* Changed gif to png in all consts
* Converted gif files to png
* Updated xmls to include pngs
@Dzordzu
Copy link
Author

Dzordzu commented Jan 2, 2025

Some background

After a recent update of the system my installation of the apache-directory-studio become unusable. I tried to build it from the source, but the effect was the same. After that I started my little investigation and found a similar issue in eclipse in 2024 (5yr ago)

System spec

  • Apache Directory Studio Version: 2.0.0.v20210717-M17
  • openjdk 17.0.13 2024-10-15
  • OpenJDK Runtime Environment 17.0.13_p11 (build 17.0.13+11)
  • OpenJDK 64-Bit Server VM 17.0.13_p11 (build 17.0.13+11, mixed mode, sharing)
  • 6.9.8-gentoo-dist
  • Apache Maven 3.9.9 (8e8579a9e76f7d015ee5ec7bfcdc97d260186937)

@Dzordzu
Copy link
Author

Dzordzu commented Jan 2, 2025

Also... reviewing it will be cumbersome. I'm sorry, but I couldn't find better way to do this

@Dzordzu Dzordzu changed the title Fixed issue with red squares [fix] Red squares instead of gif images Jan 2, 2025
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.

1 participant