Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • Characters Characters
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 56
    • Issues 56
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 3
    • Merge requests 3
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GNOMEGNOME
  • CharactersCharacters
  • Merge requests
  • !48

Use GtkPopover instead of GtkDialog to copy characters

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Matthias requested to merge matze/gnome-characters:use-popover into main Jul 14, 2020
  • Overview 14
  • Commits 3
  • Pipelines 3
  • Changes 10

There are several problems with the current modal dialog that is shown when clicking on a character:

  1. The dialog reveal is surprising and distracting because it always pops up in the middle of the parent window.
  2. The dialog cannot be closed by clicking anywhere else but the small close button on the dialog itself.
  3. As a user, I expect that the dialog closes when copying the character to the clipboard but that is not possible at the moment.
  4. The dialog layout probably violates some part of the GNOME HIG and at least to my eye it is not very attractive.
  5. The timed copy message is just eye candy but has no real value.

This commit replaces the dialog with a GtkPopover that opens at the position where the user clicked. It can be closed by either clicking somewhere else or by copying the character. The layout is a bit more condensed and dropped the "related" listbox which at least for me never showed up. As a bonus, the code is a bit shorter and easier to follow.

Before:

character-master

After:

character-using-popovers

Probably fixes or obsoletes the following issues: #43 #66 (closed) #67

/cc @bertob

Edited Feb 20, 2022 by Christopher Davis
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: use-popover