Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • P Plasma Add-ons
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 6
    • Issues 6
    • List
    • Boards
    • Service Desk
    • Milestones
  • Bugzilla
    • Bugzilla
  • Merge requests 13
    • Merge requests 13
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Plasma
  • Plasma Add-ons
  • Merge requests
  • !115

Draft: applets/notes: Slide in/out format bar

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open Mufeed Ali requested to merge fushinari/kdeplasma-addons:notes-fix into master Feb 10, 2022
  • Overview 5
  • Commits 5
  • Pipelines 8
  • Changes 1

There is a lot of room for improvement here but I don't know how to do any of it.

What's changed:

  • The format bar now slides away when plasmoid focus is lost, instead of having it turn invisible but occupying space.
  • There's now a separator above the format bar.
  • Use textMargin and padding alternately in the TextArea depending on visibility of format bar.
  • Change assumption of default SVG margins from 7% to 5%.

Issues:

  • The Notes SVG is scaled in a way that the margins are constantly changing, so the horizontalMargins and verticalMargins are always wrong. Assuming it to be any percentage of the svg's width and/or height is wrong. But I don't know what to do about it.
  • When focus is lost, the format bar slides away as expected but the text that would be below it is lost. Fixed in 12f38cf5
  • Setting textMargin has caused the scrollview to default to a position slightly off from the starting position when you have a lot of text in the sticky note. Fixed in 12f38cf5
Edited Feb 10, 2022 by Mufeed Ali
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: notes-fix