Code Monkey home page Code Monkey logo

Comments (8)

K1ngjulien avatar K1ngjulien commented on September 26, 2024 1

@vanvugt can you check the PR? that should be the last issue for the gnome 46 release :)

from dash-to-dock.

sergio-costas avatar sergio-costas commented on September 26, 2024

This is odd... neither the documentation nor the Gnome Shell source code do mark those properties as deprecated...

imagen

imagen

from dash-to-dock.

vanvugt avatar vanvugt commented on September 26, 2024
$ grep -B4 DEPRECATED *
...
--
st-scroll-view.c-    g_param_spec_object ("hscroll",
st-scroll-view.c-                         "StScrollBar",
st-scroll-view.c-                         "Horizontal scroll indicator",
st-scroll-view.c-                         ST_TYPE_SCROLL_BAR,
st-scroll-view.c:                         ST_PARAM_READABLE | G_PARAM_DEPRECATED);
--
st-scroll-view.c-    g_param_spec_object ("vscroll",
st-scroll-view.c-                         "StScrollBar",
st-scroll-view.c-                         "Vertical scroll indicator",
st-scroll-view.c-                         ST_TYPE_SCROLL_BAR,
st-scroll-view.c:                         ST_PARAM_READABLE | G_PARAM_DEPRECATED);
--
...

from dash-to-dock.

sergio-costas avatar sergio-costas commented on September 26, 2024

Oh! I was searching for something in the comments... Now it makes sense.

from dash-to-dock.

sergio-costas avatar sergio-costas commented on September 26, 2024

(also, I searched in lowercase 🤦🏾‍♂️)

from dash-to-dock.

K1ngjulien avatar K1ngjulien commented on September 26, 2024

Am I correct in seeing that this is the only place this occurs?:

dash-to-dock/dash.js

Lines 1142 to 1146 in 936b866

function ensureActorVisibleInScrollView(scrollView, actor) {
const {adjustment: vAdjustment} = scrollView.vscroll;
const {adjustment: hAdjustment} = scrollView.hscroll;
const {value: vValue0, pageSize: vPageSize, upper: vUpper} = vAdjustment;
const {value: hValue0, pageSize: hPageSize, upper: hUpper} = hAdjustment;

From what I can tell, it should be possible to just access scrollView.vAdjustment and scrollView.hAdjustment directly instead of going through vscroll and hscroll.

https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3020/diffs?commit_id=d243a9ba9320971d5ce8524ff8355b24606d9420

That seems to be why they got deprecated it in the first place:

https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3020

from dash-to-dock.

sergio-costas avatar sergio-costas commented on September 26, 2024

@K1ngjulien You are right. I left a comment in your PR, but in overall, it looks good.

from dash-to-dock.

vanvugt avatar vanvugt commented on September 26, 2024

Closed in #2185.

from dash-to-dock.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.