widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/irc_users into lp:widelands

To: mp+335898@xxxxxxxxxxxxxxxxxx
From: Notabilis <notabilis27@xxxxxx>
Date: Tue, 09 Jan 2018 19:22:39 -0000
In-reply-to: <151552533232.10282.5215383321252136414.launchpad@ackee.canonical.com>
Reply-to: mp+335898@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Additionally, I wanted to color the lines of the IRC users in gray. However, this did not work.
- A <rt><font ...> directive was printed as text.
- The existing Table::EntryRecord::set_color() method seems to be broken. At least, it does nothing and I can't find code that reads the set color.
I wouldn't mind leaving it as it is, though. Without the status symbols the lines stand out enough in my opinion.

When logging in to the metaserver, the user is greeted by the (obsolete) message "For hosting a game, ...". Maybe add a similar message "Users on IRC are unlikely to react to chat messages"? I think in the past we had some users on the forum which complained about not getting any answers.
-- 
https://code.launchpad.net/~widelands-dev/widelands/irc_users/+merge/335898
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/irc_users into lp:widelands.

References

[Merge] lp:~widelands-dev/widelands/irc_users into lp:widelands
From: Notabilis, 2018-01-09