Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 895438 - kde-apps/krdc-22.08.2: regression in scalling preventing full 1:1 scale
Summary: kde-apps/krdc-22.08.2: regression in scalling preventing full 1:1 scale
Status: RESOLVED UPSTREAM
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo KDE team
URL: https://invent.kde.org/network/krdc/-...
Whiteboard:
Keywords: UPSTREAM
Depends on:
Blocks:
 
Reported: 2023-02-19 16:48 UTC by Rafal Lalik
Modified: 2024-01-02 20:25 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch to remove regression (revoke_scaliing_regression.patch,969 bytes, patch)
2023-02-19 16:49 UTC, Rafal Lalik
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Rafal Lalik 2023-02-19 16:48:50 UTC
The krdc commit 0760c958 introduced regression in the scalling feature as described in link (I am the poster).

This patch resolves the issue. I'd recommend to apply it until the issue is resolved.

Reproducible: Always

Steps to Reproduce:
1.Open KRDC and connect to remote server. One should test it with remote larger and smaller than the local KRDC window size.
2. Be sure that the scaling slider is visible in the toolbar, add it if necessary through toolbar settings.
2. When the scalling button is OFF the remote should be fit to the window (original and desired behaviour)
3. Use slider to dynamically zoom the remote view.
Actual Results:  
For the larger remote:
1. In the left slider position we have `Fit to view`
2. For the right slider position, we have `Fit to W/H`

For the smaller remote:
1. In the left slider position we have `Fit to view`
2. For the right slider position, we have weird scaling.

Expected Results:  
For both remote smaller and larher:
1. In the left slider position we should have `Fit to view`
2. For the right slider position, we should have `1:` view

There ius discussion how to solve. I am currently in opposition to the people who submitted and merged regression. I cannot estimate when and whether this regression will be fixed.
The details are in this discussions:
https://invent.kde.org/network/krdc/-/merge_requests/41
https://invent.kde.org/network/krdc/-/merge_requests/48
Comment 1 Rafal Lalik 2023-02-19 16:49:36 UTC
Created attachment 853088 [details, diff]
Patch to remove regression
Comment 2 Andreas Sturmlechner gentoo-dev 2023-02-19 19:12:02 UTC
Gentoo is unlikely to accept a patch that is not upstream or at least on the path to being accepted upstream.