Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • Characters Characters
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 52
    • Issues 52
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 4
    • Merge requests 4
  • 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
  • !76

Avoid downscaling search result icon

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open Andrew Mayorov requested to merge encube.ul/gnome-characters:fix/blurry-search-results into main Jun 24, 2022
  • Overview 0
  • Commits 1
  • Pipelines 1
  • Changes 1

Summary

A search result icon was scaled down 2x during pixbuf export, while being rendered at 48px. This caused noticeable blurriness in shell search results under 2x display scaling. This commit prevents the downscaling.

Comparisons

Environment:

  • gnome-shell 42.2
  • gnome-characters 42.0
  • single monitor, resolution 2256 x 1504

Before @ 2x scaling:

Screenshot_from_2022-06-24_13-00-49

After @ 2x scaling:

Screenshot_from_2022-06-24_13-05-29

Before @ 1x scaling:

Screenshot_from_2022-06-24_13-03-30

After @ 1x scaling:

Screenshot_from_2022-06-24_13-04-44

Questions

  1. I suppose that ideally we need the shell to tell characters application which size it expects the icon will be. Is that right? Currently there does not seem to be a way the shell to tell that over ShellSearchProvider2 interface.

  2. This 48px icon size is currently hardcoded. I wonder what's the rationale behind choosing it?

  3. This blurriness will probably manifest again at 3x scaling or more. Should I do something about it? I was wondering if it would be feasible to find out scaling factor of the monitor the shell resides on and pick icon size depending on this factor. Though this starts to sound like a hack.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix/blurry-search-results