Applied-Software/ghostty
5
0
Fork 0
mirror of https://github.com/ghostty-org/ghostty.git synced 2025-07-16 16:56:09 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix: posToViewPort when using balanced padding enabled

Browse Source
This commit is contained in:
Lukáš Dvořák
2023-11-03 18:28:34 +01:00
parent 3eb91b8f00
commit 5224b6157c
1 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/Surface.zig
View File

@ -152,6 +152,7 @@ const DerivedConfig = struct {
vt_kam_allowed: bool,
window_padding_x: u32,
window_padding_y: u32,
window_padding_balance: bool,
pub fn init(alloc_gpa: Allocator, config: *const configpkg.Config) !DerivedConfig {
var arena = ArenaAllocator.init(alloc_gpa);
@ -174,6 +175,7 @@ const DerivedConfig = struct {
.vt_kam_allowed = config.@"vt-kam-allowed",
.window_padding_x = config.@"window-padding-x",
.window_padding_y = config.@"window-padding-y",
.window_padding_balance = config.@"window-padding-balance",
// Assignments happen sequentially so we have to do this last
// so that the memory is captured from allocs above.
@ -2107,8 +2109,13 @@ fn posToViewport(self: Surface, xpos: f64, ypos: f64) terminal.point.Viewport {
// amount from the renderer. This is a bug but realistically balanced
// padding is so small it doesn't affect selection. This may not be true
// at large font sizes...
const xpos_adjusted: f64 = xpos - @as(f64, @floatFromInt(self.padding.left));
const ypos_adjusted: f64 = ypos - @as(f64, @floatFromInt(self.padding.top));
const pad = if (self.config.window_padding_balance)
renderer.Padding.balanced(self.screen_size, self.grid_size, self.cell_size)
else
self.padding;
const xpos_adjusted: f64 = xpos - @as(f64, @floatFromInt(pad.left));
const ypos_adjusted: f64 = ypos - @as(f64, @floatFromInt(pad.top));
// xpos and ypos can be negative if while dragging, the user moves the
// mouse off the surface. Likewise, they can be larger than our surface