Project

General

Profile

提案 #1346

notify-sendにアプリケーション名を渡すようにする

Added by Shibafu Midorino 2 months ago. Updated 2 months ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
libnotify
ブランチ:
topic/1346-libnotify-appname

Description

最近リリースされたKDE Plasma 5.16から通知にアプリケーション名が表示されるようになったのですが、notify-send コマンドを経由していると「notify-send」と表示されてしまいます。(※ 添付画像参照)
内容を見ればmikutterからの通知であることは分かるのですが、できればmikutterって表示されてほしいなぁと思っています。

-a オプションを使うことでアプリケーション名を設定できるので、使ってみるのはいかがでしょうかという提案です。


Files

notify-send-app-name.jpg (45.1 KB) notify-send-app-name.jpg 参考:KDE Plasmaにおける通知表示 Shibafu Midorino, 2019-06-15 16:23
0001-notify-send-app-name.patch (1022 Bytes) 0001-notify-send-app-name.patch Shibafu Midorino, 2019-06-15 16:23
0001-libnotify-app-name.patch (960 Bytes) 0001-libnotify-app-name.patch Shibafu Midorino, 2019-06-16 00:06

Related issues

Related to 提案 #1319: libnotifyプラグインの、rnotify.rbを削除する終了Actions

Associated revisions

Revision da4c2725 (diff)
Added by Shibafu Midorino 2 months ago

libnotifyにアプリケーション名を渡す refs #1346

Revision 38ea0c9a (diff)
Added by toshi_a 初音 2 months ago

Revert "libnotifyにアプリケーション名を渡す refs #1346"

This reverts commit da4c27256652a96881190970b8b178a2a024615c.

History

#1

Updated by toshi_a 初音 2 months ago

あれっ、おかしいな。すでにnotify-sendコマンドの利用は止めているはずなのに…?

#2

Updated by toshi_a 初音 2 months ago

この提案自体には賛成です。

notify-sendを使わずに通知するやつ、時空の間に吸い込まれて消滅していたようなので、ちょっと整理します。
基本的にはそれをmergeするつもりなので、このパッチはそのまま使うことは出来ません。実装は多分可能だと思います。

#3

Updated by toshi_a 初音 2 months ago

  • Related to 提案 #1319: libnotifyプラグインの、rnotify.rbを削除する added
#4

Updated by Shibafu Midorino 2 months ago

なんか通知回りの話あったような……とは思っていたんですが、まさかマージされていなかったとは。

ということで、libnotify 対応版のパッチをさっと書いてみました。

#5

Updated by toshi_a 初音 2 months ago

  • Status changed from 分類待ち to パッチ適用待ち
  • Assignee set to toshi_a 初音
#6

Updated by toshi_a 初音 2 months ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • Assignee changed from toshi_a 初音 to Shibafu Midorino
  • Target version set to 3.9
  • プラグイン名 set to libnotify
  • ブランチ set to topic/1346-libnotify-appname

ということなのでささっとcommitしてみました

#7

Updated by Shibafu Midorino 2 months ago

こんな声があったのでちょっと追加で調べ物をしていました。

@yuntan_t@mstdn.maud.io
ついでにアプリのアイコンも…
https://mstdn.maud.io/@yuntan_t/102276216734533768

ここで言うアプリのアイコンというのは、 このトゥート に添付されている画像の左上のように、アプリケーション名の左に表示されるアイコンのことです。

このアイコンは何を基に表示されてるかというと、通知に "desktop-entry" Hint として指定されたデスクトップエントリを検索して、Iconで指定されている画像を出しているようです。
("desktop-entry" Hint はこれ以外にも、通知サーバーの実装によりますがアプリごとに通知をグループ化したりするのに使われているみたいですね。)

mikutterでも、パッケージによってはデスクトップエントリが含まれていたような気がしますので、可能であれば設定してあげたい……と思ったのですが、どうも libnotify gem では notify_notification_set_hint_string を適切に呼び出すメソッドがプライベートになっているようです。残念。

----
(調査中のトゥートをメモ的にリンクしておきます)
https://social.mikutter.hachune.net/@shibafu528/102276472078168904
https://social.mikutter.hachune.net/@shibafu528/102276476169586171
https://social.mikutter.hachune.net/@shibafu528/102276484112682836

#8

Updated by Shibafu Midorino 2 months ago

  • Status changed from レビュー待ち to マージ待ち

コミットについては確認しました。問題ありません。

#9

Updated by toshi_a 初音 2 months ago

  • Status changed from マージ待ち to 終了

mergeしました

Also available in: Atom PDF