incubator-wave-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject Re: Review Request: Implementing the feature "New wave with the participants of the current wave"
Date Fri, 12 Oct 2012 11:07:35 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Oct. 12, 2012, 11:07 a.m.)

Review request for wave.


* Moved the logic of adding a participant set to the OpBasedWavelet class and added a test
case for that method
* Updated the buttons to look the same as the search buttons
* Added a dialog for selecting which participant to add. See screenshot

Please review.


I've started to create the functionality me and my friends used a lot in Google Wave - to
be able to create a new wave with the participants from an open wave. (I could not find a
JIRA for this)

The functionality of the patch is working as intended. There is currently a few things that
I would like some input on:

UI (See the screenshot)
1. The icon I use is currently only a placeholder icon. Is anyone able to create a better
icon or any ideas of how it can look and I can try to arrange one? Or is it better to just
display plain text?
2. Is it OK to add this functionality as an icon/text beside the "Add participant" button
for now or should it be placed in a submenu instead that can eventually contain more functionality?
Maybe at least the button should be placed to the far right edge of the participant panel?

3. I currently just added the participant addition code into the install method of the StageTwoProvider.
It feels like it might not be the best spot. Any feedback on where it could be placed?
4. In the WaveCreationEvent class I only create one "CREATE_NEW_WAVE_WITH_PARTICIPANTS"-event
where I update the participant pointer. Maybe it is safer to create a new event for every
5. Any other implementation feedback?


Diffs (updated)

  src/org/waveprotocol/box/webclient/client/ 7a6c8ec 
  src/org/waveprotocol/box/webclient/client/ 9d83269 
  src/org/waveprotocol/box/webclient/client/ 863ae6c 
  src/org/waveprotocol/box/webclient/client/events/ 95c317b 
  src/org/waveprotocol/box/webclient/client/events/ adc57f9 
  src/org/waveprotocol/wave/client/ 5a8de4d 
  src/org/waveprotocol/wave/client/wavepanel/impl/edit/ a6eeb53

  src/org/waveprotocol/wave/client/wavepanel/impl/edit/ PRE-CREATION

  src/org/waveprotocol/wave/client/wavepanel/impl/edit/ParticipantSelectorWidget.ui.xml PRE-CREATION

  src/org/waveprotocol/wave/client/wavepanel/impl/edit/ PRE-CREATION

  src/org/waveprotocol/wave/client/wavepanel/impl/edit/ParticipantWidget.ui.xml PRE-CREATION

  src/org/waveprotocol/wave/client/wavepanel/view/ c770b36 
  src/org/waveprotocol/wave/client/wavepanel/view/dom/ 64e7f79 
  src/org/waveprotocol/wave/client/wavepanel/view/dom/ e848c5f 
  src/org/waveprotocol/wave/client/wavepanel/view/dom/full/Participants.css 66f1836 
  src/org/waveprotocol/wave/client/wavepanel/view/dom/full/ 735ec04

  src/org/waveprotocol/wave/client/wavepanel/view/dom/full/ ba8bc7a 
  src/org/waveprotocol/wave/concurrencycontrol/wave/ e87b1e0 
  src/org/waveprotocol/wave/model/conversation/ cff46ed 
  src/org/waveprotocol/wave/model/conversation/ 62c5364 
  src/org/waveprotocol/wave/model/wave/ a463d45 
  src/org/waveprotocol/wave/model/wave/opbased/ e99067c 
  test/org/waveprotocol/wave/model/wave/opbased/ 1ffd8f3 


Testing (updated)

Tested manually on a local server both the implementation and the button placement/adjustment
when adding/removing participant and shrinking the window size
Added one test case for adding a set of participants to a wave

Screenshots (updated)



  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message