GithubHelp home page GithubHelp logo

Comments (7)

toolstack avatar toolstack commented on August 19, 2024

I assume you haven't change the "Limit note width" setting? If so then this is mostly expected, though I agree that it should probably be able to go smaller than the fixed width, just not larger.

from folio.

toolstack avatar toolstack commented on August 19, 2024

Ok, so I looked into this and it appears to be expected behaviour with GTK4/Adwaita apps and the SourceView control that Folio uses.

Basically "Limit note width" is mislabelled, it should be "Set fixed note width".

Trying to auto re-size the widget would go against several of the GTK4/Adwaita principals so I'm not going to attempt that as it will just break again in the future more than likely.

Instead I'm going to commit two changes to resolve this issue:

  • Relabel the preference
  • Add a new pref to allow the user to set the fixed width size, at the moment it is 720, which will be the default

from folio.

alice-mkh avatar alice-mkh commented on August 19, 2024

libadwaita maintainer here, someone pinged me to take a look at this issue and I have no idea what principles you mean.

  1. AdwClampScrollable
  2. While looking at it I noticed that there's a toast overlay inside the scrolled window?.. Why? Put it right into the window instead
  3. GtkSourceView inside a GtkViewport (or AdwToastOverlay, for that matter) is not supported, so you likely broken scrolling on large notes right now. AdwClampScrollable should fix that too

from folio.

toolstack avatar toolstack commented on August 19, 2024

@alice-mkh thanks, I'm new to GTK/libadwaita but from what I saw it looks like monitoring window size changes is discouraged, which seemed to be the only way to reset the size of the sourceview widget if it was set to a fixed width instead of letting it fill the window.

I'll take a look at AdwClampScrollable.

from folio.

alice-mkh avatar alice-mkh commented on August 19, 2024

You don't need to monitor window sizes here, no.

if it was set to a fixed width

Right, don't do this. And you have to get rid of that toast overlay too (and of viewport if you added it manually, clamp scrollable replaces that)

from folio.

toolstack avatar toolstack commented on August 19, 2024

The primary notes area, by default, has a fixed with, so I can't just undo that easily without breaking existing behaviour. As such I've implemented an alternate solution that works for Folio.

I'll probably look at changing the default to non-fixed width in a future release though.

I took a quick look at clamp scrollable and don't think it will fit within the app at the moment, but I do have to do a major change at some point as Folio currently uses a lot of deprecated widgets, so I'll look more at it when I get to that.

from folio.

alice-mkh avatar alice-mkh commented on August 19, 2024

The primary notes area, by default, has a fixed with

Yes, which results in horizontal scrolling right with default window size. That's a bug, not something to preserve.

from folio.

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.