Added a UI field to display group count in the groups floater, plus code to update it. This adresses BUG-81.


Submitter:	Lance Corrimal	Reviewers
Branch:		Groups:	viewer
Bugs:	BUG-81	People:
Change Number:	None	Repository:	viewer-development

Description:

This patch adds the number of groups you're in to the bottom of the group list.

Testing Done:

Testing done in the latest dolphin viewer beta, and in a recent viewer-development fork. The actual implementation has been in Marine Kelleys RLV for a while, and has been adapted with her permission.

Expand All

Ship It!

MartinRJ Fayray Sept. 12, 2012, 11:25 p.m. (Sept. 12, 2012, 11:25 p.m.)

Just tested with a modded 3.4.0 viewer-release (from revision 7a056d7afeb2 - https://bitbucket.org/lindenlab/viewer-release/changeset/7a056d7afeb246435e31d06cee2a9a85eac82476).
Works as expected.
Ship it!

indra/newview/skins/default/xui/en/panel_people.xml (Diff revision 1)

We think in would be clearer to use a message like:

You belong to [COUNT] groups, and can join [REMAINING] more.

where REMAINING is the maximum minus COUNT.

Lance Corrimal Sept. 13, 2012, 2:06 p.m. (Sept. 13, 2012, 2:06 p.m.)

I have a new version in https://bitbucket.org/LanceCorrimal/storm-1910/changeset/275e5ab93bfa but for some reason CR does not let me upload a diff...

Lance Corrimal Sept. 13, 2012, 11:36 p.m. (Sept. 13, 2012, 11:36 p.m.)

Latest version: http://dolphinsource.eregion.de/storm-1910/changeset/72f674a51acc/
Why can't I update the diff in this CR request anymore?

status changed from pending to discarded

Description:

For some reason I cannot upload a new patch... Going to create a new CR instead.

status changed from discarded to pending

Diff
- added Diff r2

Description:

diff updated to confirm with Oz' change request

Ship It!

status changed from pending to submitted

You have a pending review.

Other reviews

Your comment

Review Board 1.6.11

This change has been marked as submitted.