Skip to content

Fixed icewm configuration using setup from yast#1931

Merged
lslezak merged 7 commits intomasterfrom
fix_icewm_setup
Feb 6, 2025
Merged

Fixed icewm configuration using setup from yast#1931
lslezak merged 7 commits intomasterfrom
fix_icewm_setup

Conversation

@mchf
Copy link
Contributor

@mchf mchf commented Jan 22, 2025

Problem

In running Agama installer user was able to open IceWM's main menu using Win key

Solution

Taken preferences.yast2 from yast-theme package

Note: the Makefile patch was taken from @lslezak https://github.com/agama-project/agama/compare/efi_locale?expand=1#diff-3814de00178ccbebe225da81ee5f56663518b579103d005782925a8b014491ca

Notes

  • The default IceWM shortcut for starting xterm is Ctrl+Alt+T

  • Additionally fixed crashing xterm when displaying the xterm menu (via Ctrl + click)
    agama-xterm

  • I (@lslezak) have enabled some IceWM features in the config file. Enable displaying the window title, the maximize/minimize buttons and the window actions menu in the title bar.

    YaST uses popup windows where these features are not wanted, but Agama displays popups in the main browser window (so technically they are not popups and we do not use any alert() or similar functions). And these features might be useful when running xterm debug window. See the screenshot above.

Testing

  • Tested manually

@mchf mchf requested a review from imobachgs January 22, 2025 09:52
@coveralls
Copy link

coveralls commented Jan 22, 2025

Pull Request Test Coverage Report for Build 13072017008

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 70.905%

Totals Coverage Status
Change from base Build 13036177704: 0.0%
Covered Lines: 17291
Relevant Lines: 24386

💛 - Coveralls

Copy link
Contributor

@imobachgs imobachgs left a comment

Choose a reason for hiding this comment

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

Please, do not merge yet until beta 1 is released.

Copy link
Contributor

@ancorgs ancorgs left a comment

Choose a reason for hiding this comment

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

LGTM, now with the added fonts

...instead of installing extra font package (23MB compressed size)
Enable displaying the window title, the maximize/minimize buttons
and the window actions menu in the title bar.

YaST uses popup windows where these features are not wanted,
but Agama displays popups in the main browser window (so technically
they are not popups and we do not use any `alert()` or similar
functions). And these features might be useful when running xterm
debug window.
@lslezak lslezak merged commit 000afa8 into master Feb 6, 2025
6 of 7 checks passed
@lslezak lslezak deleted the fix_icewm_setup branch February 6, 2025 14:47
@imobachgs imobachgs mentioned this pull request Feb 26, 2025
imobachgs added a commit that referenced this pull request Feb 26, 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.

5 participants