Skip to content

fixed #13909 / refs #13914 - some (enforced) language handling fixes for the GUI #7570

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

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

firewave
Copy link
Collaborator

@firewave firewave commented Jun 3, 2025

No description provided.

@firewave firewave changed the title fixed #13909 fixed #13909/#12841 Jun 3, 2025
@firewave firewave changed the title fixed #13909/#12841 fixed #13909/#12841 / refs #13914 Jun 3, 2025
@firewave firewave force-pushed the import-gui branch 10 times, most recently from f85cc26 to 2b9d467 Compare June 7, 2025 14:19
@firewave

This comment was marked as resolved.

@firewave firewave changed the title fixed #13909/#12841 / refs #13914 fixed #13909 / refs #13914 Jun 10, 2025
@firewave firewave changed the title fixed #13909 / refs #13914 fixed #13909 / refs #13914 - fixed (enforced) language handling in the GUI Jun 10, 2025
@firewave firewave changed the title fixed #13909 / refs #13914 - fixed (enforced) language handling in the GUI fixed #13909 / refs #13914 - some (enforced) language handling fixes for the GUI Jun 10, 2025
namespace frontend
{}
{
void applyLang(std::list<FileSettings> &fileSettings, const Settings &settings, Standards::Language enforcedLang);
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still needs documentation - and unit test.

@firewave
Copy link
Collaborator Author

This makes me wonder if markup files are handled properly in the GUI. But not something to explicitly look into as it should be implicitly be fixed as we move towards the shared executor code.

@firewave
Copy link
Collaborator Author

Requires #7598 to be merged first.

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