Linux support#136
Conversation
remove if you don't want it in the repo ig
…ot resizing not sure if that even is impl somewhere
unsecretised
left a comment
unsecretised
left a comment
There was a problem hiding this comment.
Just that one change and we shld be good to go
|
HELL YEAH |
|
(windows built was broken before) |
| } | ||
|
|
||
| pub fn contents_style(theme: &ConfigTheme) -> container::Style { | ||
| pub fn contents_style(_theme: &ConfigTheme) -> container::Style { |
There was a problem hiding this comment.
could this be changed to 2 different functions for windows / macos?
There was a problem hiding this comment.
i don't really know why i even changed the text_color and radius stuff. just reverted it and made it None so it should work the same way for both
|
Ok, the hotkey on macos isn't working for some reason. @Mnem42 can you check windows support |
will look into it. |
|
Doesn't compile on windows, but nothing too major, and I'll just add them as a commit (since none of them are very related to the changes you made) 
(btw, sorry about causing it to be a rather horrible merge conflict) |
|
@kybe236 fine to merge? |
|
|
I think before the merge you should do the clean up first. I'll wait for the windows support branch to be ready and then we'll merge the windows support branch once. Then we'll go ahead and add support to it slowly. |
|
shall be fine to merge now |
4 participants
linux support :-)