After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 706353 - About box changes size when license button is toggled on, link to website is wrong
About box changes size when license button is toggled on, link to website is ...
Status: RESOLVED FIXED
Product: gnome-music
Classification: Applications
Component: general
unspecified
Other Linux
: Normal minor
: ---
Assigned To: gnome-music-maint
gnome-music-maint
Depends on:
Blocks:
 
 
Reported: 2013-08-20 03:06 UTC by Arnel Borja
Modified: 2013-08-22 16:37 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fix the upstream URL in the "about" dialog (1.77 KB, patch)
2013-08-21 09:13 UTC, Mathieu Bridon
committed Details | Review
Fix the upstream URL in the "about" dialog (1.82 KB, patch)
2013-08-21 09:38 UTC, Arnel Borja
committed Details | Review
fix formatting in Credits and License (5.53 KB, patch)
2013-08-22 15:10 UTC, Paul Martin
committed Details | Review
Fix formatting in About dialog (4.79 KB, patch)
2013-08-22 16:36 UTC, Arnel Borja
committed Details | Review

Description Arnel Borja 2013-08-20 03:06:06 UTC
The GNOME Wiki has some changes a few days ago, so the link to the "website" of GNOME Music is now "https://wiki.gnome.org/Apps/Music".

Also the about box changes size when the license is shown, it doesn't happen with Totem for example. We might need to lessen the words per line (though I don't think that's a good idea - this part is translatable so it might happen to translations too, and the font there is not monospace so it would be hard to get it right).
Comment 1 Arnel Borja 2013-08-20 03:24:51 UTC
Another improvement would be to alphabetize the designers. The maintainers and contributors are currently alphabetized.
Comment 2 Arnel Borja 2013-08-21 08:35:45 UTC
I did some changes in bug #706457 that affects this.
Comment 3 Mathieu Bridon 2013-08-21 09:13:15 UTC
Created attachment 252513 [details] [review]
Fix the upstream URL in the "about" dialog
Comment 4 Arnel Borja 2013-08-21 09:37:50 UTC
Comment on attachment 252513 [details] [review]
Fix the upstream URL in the "about" dialog

The following fix has been pushed:
36b15bb Fix the upstream URL in the "about" dialog
Comment 5 Arnel Borja 2013-08-21 09:38:03 UTC
Created attachment 252515 [details] [review]
Fix the upstream URL in the "about" dialog

This in fact fixes the URL in the configure.ac file, and then reuses
that in the "about" dialog.

If it needs to change in the future, then it can be changed only in one
place.
Comment 6 Arnel Borja 2013-08-21 10:04:50 UTC
Now what remains are the following fixes for the About dialog:

- Add a newline after the Contributors list to separate it with the Designers list
- List designers alphabetically
- Remove newlines in the license (GNU follows a convention of two spaces after dots)
- Enable wrapping of license
Comment 7 Paul Martin 2013-08-22 15:10:39 UTC
Created attachment 252762 [details] [review]
fix formatting in Credits and License
Comment 8 Arnel Borja 2013-08-22 16:36:13 UTC
The following fix has been pushed:
2199f3a Fix formatting in About dialog

I modified the patch:
- I removed a line in the patch which prevents the dialog to be modal (hint property)
- Added newlines per section and removed some newlines missed
- Removed my notes to the commit message and a comment

Thanks for fixing this bug! :D
Comment 9 Arnel Borja 2013-08-22 16:36:18 UTC
Created attachment 252773 [details] [review]
Fix formatting in About dialog

- Add a newline after the Contributors list to separate it with the
  Designers list
- List designers alphabetically
- Remove newlines in the license
- Enable wrapping of license