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

Two tiny suggestions about c++17 and fa_solid_900.h, and a problem concerning X close button. #5

Closed
sirlis opened this issue Oct 10, 2023 · 2 comments · Fixed by #10
Assignees
Labels
documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers question Further information is requested

Comments

@sirlis
Copy link

sirlis commented Oct 10, 2023

  1. Line 455 of ImGuiNotify.hpp:
inline std::vector<ImGuiToast> notifications;

requires stdc++17, which I suggest to explicity informed in the README.

  1. Line 26 of ImGuiNotify.hpp:
#include "fa_solid_900.h"

don't know why I built with failure. Win11 with cmake, opengl, glfw, glad, imgui. I suggest remove it, seems to be of no use. Or do I made a mistake?

  1. [Problem] My render result is as follows:

image

don't know why my X close button is so ugly.

When switching from FONT_ICON_FILE_NAME_FAS to FONT_ICON_FILE_NAME_FAR, the close button become ?.

@sirlis sirlis changed the title Two tiny suggestions, c++17 and fa_solid_900.h Two tiny suggestions, c++17 and fa_solid_900.h. A problem concerning X close button. Oct 10, 2023
@sirlis sirlis changed the title Two tiny suggestions, c++17 and fa_solid_900.h. A problem concerning X close button. Two tiny suggestions about c++17 and fa_solid_900.h, and a problem concerning X close button. Oct 10, 2023
@TyomaVader TyomaVader self-assigned this Oct 11, 2023
@TyomaVader
Copy link
Owner

Thanks for the issue

  1. README will be updated.
  2. Works for me on Win11, but yes, it's a useless file, .ttf is now used instead. Thanks for noticing!
  3. Looks like all styles are set to default for you, have you disabled the EmbraceTheDarkness theme ? Make sure that colors[ImGuiCol_Button], colors[ImGuiCol_ButtonHovered] and colors[ImGuiCol_ButtonActive] are set to the required colors. The ? when using FONT_ICON_FILE_NAME_FAR is probably an issue with FontAwesome6, a few other icons are broken actually.

@TyomaVader TyomaVader added documentation Improvements or additions to documentation enhancement New feature or request question Further information is requested labels Oct 11, 2023
@sirlis
Copy link
Author

sirlis commented Oct 12, 2023

Thanks for the issue

  1. README will be updated.
  2. Works for me on Win11, but yes, it's a useless file, .ttf is now used instead. Thanks for noticing!
  3. Looks like all styles are set to default for you, have you disabled the EmbraceTheDarkness theme ? Make sure that colors[ImGuiCol_Button], colors[ImGuiCol_ButtonHovered] and colors[ImGuiCol_ButtonActive] are set to the required colors. The ? when using FONT_ICON_FILE_NAME_FAR is probably an issue with FontAwesome6, a few other icons are broken actually.

Thanks for your kind response. I will check if my theme is correct. Basically, I do not modify any theme related thing. I just introduced the original imgui into my project. Anyway, not a big deal.

@sirlis sirlis closed this as completed Oct 12, 2023
@TyomaVader TyomaVader added the good first issue Good for newcomers label Feb 3, 2024
@TyomaVader TyomaVader mentioned this issue Jul 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants