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

rootston module: better defaults #33085

Merged
merged 2 commits into from Dec 27, 2017
Merged

rootston module: better defaults #33085

merged 2 commits into from Dec 27, 2017

Conversation

ghost
Copy link

@ghost ghost commented Dec 26, 2017

Motivation for this change

cc @primeos

  • Don't use X11's urxvt, because it has no decoration and so no movable with mouse.
    Use weston-terminal instead of it. If you don't like weston-terminal, swap with termite (another wayland native). I mainly put weston-terminal because orbment and way-cooler use it by default.
    Looks like wlroots author even uses gnome-terminal Add fractional scaling support swaywm/wlroots#488 (comment)
  • Don't use half-working dmenu
    rofi works great for me as been suggested at Wayland menu for Sway? swaywm/sway#1367 (comment)
  • Use dconf like in sway and way-cooler modules
    Useful to testing e.g. scaling in rootston. To ensure that all GTK3 programs will use configured font size and other settings.
  • extraSessionCommands is useful e.g. for enabling qtwayland without poking with shell and environment.

BTW The reason why urxvt and dmenu are used in sway module is because default Sway configuration uses them.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@primeos
Copy link
Member

primeos commented Dec 26, 2017

Looks way better than my buggy draft, thanks 😄
Unfortunately I can't test it ATM (on the 34C3 without my Wayland system) but the patch looks fine (thanks for the explanations btw), the tests are ok and I assumed you've tested it so there should be no reason not to merge it.

Feel free to add yourself as a maintainer if you want to maintain this module as well 😉 (if not I'll merge it tomorrow).

@ghost
Copy link
Author

ghost commented Dec 27, 2017

@primeos Done. Thanks for packaging wlroots and rootston! It makes it simple to track their progress. After I've seen that the fractional scaling works fine, I have a high hopes for forthcoming Sway-on-wlroots and will definately switch to it from my current i3 setup 😄

@primeos primeos merged commit b15eea6 into NixOS:master Dec 27, 2017
@ghost ghost deleted the rootston branch December 27, 2017 15:39
# Define a keymap (US QWERTY is the default)
export XKB_DEFAULT_LAYOUT=de,us
export XKB_DEFAULT_VARIANT=nodeadkeys
export XKB_DEFAULT_OPTIONS=grp:alt_shift_toggle,caps:escape
Copy link

Choose a reason for hiding this comment

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

The preferred way to do this is in the config file.

Copy link
Author

Choose a reason for hiding this comment

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

Copy link

Choose a reason for hiding this comment

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

rootston != sway

Copy link
Author

Choose a reason for hiding this comment

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

@emersion Then feel free to fix it :-)

Copy link
Member

@primeos primeos Apr 3, 2018

Choose a reason for hiding this comment

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

Hi :) I think there was a bit of a misunderstanding.

@emersion I think you mean something like this (didn't check the syntax yet, just had a quick look at swaywm/wlroots#341)?:

[keyboard]
layout = de,us
variant = nodeadkeys
options = grp:alt_shift_toggle,caps:escape

I'll have a closer look (maybe open an upstream issue) and make a PR later. Thank you for letting us know how we can improve this module (wasn't aware that this is possible) 😄.

@gnidorah I assume (might be completely wrong) you where a bit "irritated" by the first comment (at least I initially was). @emersion is the main(?) wlroots developer and as I interpret his comment (now), is that he just wanted to help us. (Edit: And you're right these options where not part of the config at the time of this PR, i.e. it's the new "preferred" way.)

Copy link

Choose a reason for hiding this comment

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

Yeah, sorry, I really didn't mean to be irritating. I just stopped by and saw you're still using the Sway way, which is supported for legacy reasons. It's no big deal in the end, feel free to do whatever you want. :)

Copy link
Author

Choose a reason for hiding this comment

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

@emersion @primeos Ah, got it, thank you guys!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants