! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
xfce4-about - dark blue e-mail addresses on dark themes
Status:
RESOLVED: FIXED
Product:
Libxfce4ui
Component:
General

Comments

Description Spass 2018-08-29 17:59:07 CEST
Created attachment 7915 
screenshot

Hello. I'm using a dark theme on my Xfce (Xubuntu 18.04 and 18.10) and xfce4-about Credits tab does not look good. Blue color on the e-mail addresses is hardcoded:
https://git.xfce.org/xfce/libxfce4ui/tree/xfce4-about/main.c#n220

Since those addresses are not even a link, it would be great if that color was more visible. Even better if that color could inherit a "link" color from a GTK theme.
Comment 1 Andre Miranda editbugs 2018-08-31 05:40:21 CEST
I tried gtk_text_buffer_insert_markup but pango supports only real basic stuff: https://developer.gnome.org/pango/stable/PangoMarkupFormat.html

The options are:
- Remove the blue color, since it's not a link. Anyway, direct e-mail contact is not the preferred way to get in touch, mostly like the last resort. Users should use xfce4-dev@xfce.org, bugzilla or#xfce-dev. In any case, one always copy and paste the e-mail address from this dialog.
- Get the text color from current theme. Not so easy, possibly we also need to watch for themes changes. Still the link won't do anything when clicked, no idea how to do that with GtkTextView/GtkTextBuffer.
- Restructure the whole dialog with GtkLabels (they support <a></a>) arranged in a GtkGrid.

The first option is my preferred, if agreed I can prepare a patch (do not expect me to implement the other two options).
Comment 2 vladimir 2018-08-31 08:21:24 CEST
(In reply to Andre Miranda from comment #1)
> Я попытался gtk_text_buffer_insert_markup но панго поддерживает только
> реальные базовые вещи:
> https://developer.gnome.org/pango/stable/PangoMarkupFormat.html
> 
> Варианты::
> - Удалить синий цвет, так как это не ссылка. Во всяком случае, прямой
> контакт по электронной почте не является предпочтительным способом, чтобы
> войти в контакт, в основном, как в крайнем случае. Пользователи должны
> использовать xfce4-dev@xfce.org, bugzilla или#xfce-dev. В любом случае,
> всегда нужно скопировать и вставить адрес электронной почты из этого диалога.
> - Получить цвет текста из текущей темы. Не так просто, возможно, нам также
> нужно следить за изменениями тем. Тем не менее, ссылка ничего не будет
> делать при нажатии, понятия не имеет, как это сделать с
> GtkTextView/GtkTextBuffer.
> - Реструктурировать весь диалог с GtkLabels (они поддерживают <a></a>),
> расположенный в GtkGrid.
> 
> Первый вариант-мой предпочтительный, если я могу подготовить патч (не
> ожидайте, что я буду реализовывать два других варианта).

Isn't there in the version of xfce to indicate 4.13? In my opinion the current version of xfce is not 4.12....
Comment 3 vladimir 2018-08-31 08:23:49 CEST
Created attachment 7922 
screenshot
Comment 4 Andre Miranda editbugs 2018-09-01 03:06:30 CEST
I asked this once at #xfce-dev, someone told that historically this dialog was not updated during dev releases, previously I don't think it was a problem, just 4.14 is taking longer than we have wished.
Comment 5 Skunnyk editbugs 2018-09-23 19:35:19 CEST
Yes, we can simply remove the "foreground blue" attribute to the email list.
Comment 6 Skunnyk editbugs 2018-09-23 19:35:45 CEST
Created attachment 7996 
without foreground blue
Comment 7 Andre Miranda editbugs 2018-09-23 20:04:10 CEST
Created attachment 7997 
remove-email-tag.patch

I went a bit further and removed all styling of emails, I don't like that underline, still looks clickable.
Comment 8 Git Bot editbugs 2019-04-16 23:33:00 CEST
Andre Miranda referenced this bugreport in commit b36d27fb4b55a173ef931681855458c75e698e27

Bug #14638: Remove styling of emails in about dialog

https://git.xfce.org/xfce/libxfce4ui/commit?id=b36d27fb4b55a173ef931681855458c75e698e27

Bug #14638

Reported by:
Spass
Reported on: 2018-08-29
Last modified on: 2019-04-17

People

Assignee:
Nick Schermer
CC List:
5 users

Version

Version:
4.12.1

Attachments

screenshot (89.08 KB, image/png)
2018-08-29 17:59 CEST , Spass
no flags
screenshot (246.91 KB, image/png)
2018-08-31 08:23 CEST , vladimir
no flags
without foreground blue (60.07 KB, image/png)
2018-09-23 19:35 CEST , Skunnyk
no flags
remove-email-tag.patch (2.54 KB, patch)
2018-09-23 20:04 CEST , Andre Miranda
no flags

Additional information