-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3960/
-----------------------------------------------------------
(Updated 2012-03-17 20:25:34.239494)
Review request for wave.
Changes
-------
Please review patch. Thanks!
* Created GadgetInfoProvider interface and GadgetInfoProviderImpl that
implements it
* Created manual injection of the GadgetInfoProvider in WebClient.java - I
think this might not be exactly as you meant @Yuri, maybe you can point me in
the right direction if you don't think this solution is good? I could not find
any similar example in the current web client source code.
* Added proper error handling and error logging
* Updated according to the comments
Summary
-------
* Added more gadgets to the gadget list
* Made the gadget list scrollable and filterable through a text box and a drop
down box for categories.
* The filtering looks at both the name of the gadget and its description.
Author could also be added as a searchable property (?)
* Filter the result directly in the scrollable gadget list
* Marking the top filtered search as selected with gray background and made it
possible to choose that gadget by pressing enter
* Change the default focus to the filter box to allow the user to quickly
select a gadget from the list by filter it out and pressing enter
This addresses bug wave-319.
https://issues.apache.org/jira/browse/wave-319
Diffs (updated)
-----
src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/GadgetSelectorWidget.ui.xml
cc6b73e
src/org/waveprotocol/wave/client/widget/popup/CenterPopupPositioner.java
555f32e
src/org/waveprotocol/wave/client/widget/popup/PositionUtil.java 5e3bab4
test/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/GadgetInfoProviderTest.java
PRE-CREATION
src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/GadgetInfoWidget.java
97611b4
src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/GadgetInfoWidget.ui.xml
c8b7a81
src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/GadgetSelectorWidget.java
ccbcdae
src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/GadgetInfoProviderImpl.java
PRE-CREATION
src/org/waveprotocol/box/server/ServerMain.java 13a2d55
src/org/waveprotocol/box/server/persistence/file/FileUtils.java 538e6e3
src/org/waveprotocol/box/server/rpc/GadgetProviderServlet.java PRE-CREATION
src/org/waveprotocol/box/webclient/client/WebClient.java c87b7cc
src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/gadget/GadgetInfoProvider.java
PRE-CREATION
jsongadgets.json PRE-CREATION
Diff: https://reviews.apache.org/r/3960/diff
Testing
-------
Tested locally.
Successfully run all junit test (Except
PerUserWaveViewSubscriberTest::testGetPerUserWaveView and
WaveServerTest::testWaveletNotification that fails even before this patch)
Tested all included gadgets
Thanks,
rocklund
