Bumped version of rmpv in neovide to 1.0.0 to match nvim-rs.
Also, fixed a warning where a variable need not be mut.
Co-authored-by: Oliver Krylow <o.krylow@isp-insoft.de>
In Linux, custom changes to the $PATH are usually defined in ~/.profile,
~/.bash_profile, ~/.zprofile, etc. These files are only loaded by a
'login shell', which can be explicitly invoked with the -l flag. Most
shells (bash, dash, zsh, fish) support this -l flag. This commit uses
the $SHELL env var to invoke the user's default shell with the -l flag to
force a login shell. This should properly load any custom $PATH settings
defined by the WSL user. This is a potential fix for #693
Avoids
```
warning: unused variable: `points_per_inch`
--> src/renderer/fonts/font_options.rs:94:9
|
94 | let points_per_inch = 72.0;
| ^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_points_per_inch`
|
= note: `#[warn(unused_variables)]` on by default
```
on macOS, which is innocuous but has been bugging me for a while.
Using `has("nvim-0.4")` check seems more robust than using a regex to
parse the version string. This also means regex is no longer a runtime
dependency.
This is a rather heavy dependency that's easily replaceable using only
the standard library. This brings the number of dependencies down from
368 to 362 (on Linux).
Paired values, like width/height now using tuples and Dimensions.
`Dimensions` struct implements traits for multiplication and division,
which allows us to scale grid size to physical and back just by
multiplying `Dimensions` instead of each variable.
Also, added trait for multiplication (u64, u64) by Dimensions.
For the same purpose as above.
Setting initial window size moved into draw_frame, and called
after fonts are loaded. So initial window size matches requested
geometry.
This also fixes missing startup screen, as UiCommand::Resize is not
called on startup.
Providing a wrong font was causing neovide to retry every time to load an
inexistent font, instead just trying to load once at it's setup should
be enough
* trying to make app_id and class exchangeable
* still fails to hand over the argument
* make both app_id and class be set by command line arguments
* added missing cfg for class
* Changes as requested. Putting the os check in a block however doesn't
work for me
* getting rid of superfluous to_string()
* x11-class argument gets ignored
* changed argument name
* Update src/cmd_line.rs
* changed faulty unwrap
* added missing comma
* fixed wrong case
* fixed class
* make cargo fmt happy
* exchanged String::from with String::new
* make both attributes also overrideable via env::var
Co-authored-by: partizan <serg.partizan@gmail.com>
* refactor(keyboard_manager): Refactor KeyboardManager
This refactoring changes nothing in key handling behaviour, but moves
some code out of handle_event, to make thigs clearer.
* style(keyboard_manager): Move private functions to bottom
set_inner_size was using LogicalSize from current physical size,
it was working fine only for scaling=1.
Removing this leaves unmaximizing to winit, which does it correctly.
Also, replaced inverse scaling (like 1 / scaling) with natural scaling.
I think easier to work with natural scaling factor (like 1..2)
and multiplication, instead of inverse scaling factor and division.
Refs #771