Vojtech Szocs has posted comments on this change.
Change subject: webadmin: clicking system tree doesn't update main tab
..
Patch Set 1:
(1 comment)
https://gerrit.ovirt.org/#/c/42238/1/frontend/webadmin/modules/gwt-common/src
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: made DynamicMessages accessible in
uicommonweb
..
Patch Set 5: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/41947
To unsubscri
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: made DynamicMessages accessible in
uicommonweb
..
Patch Set 4: Code-Review+1
(1 comment)
https://gerrit.ovirt.org/#/c/41947/4/frontend/web
Vojtech Szocs has submitted this change and it was merged.
Change subject: core: Stop overriding id of new permission in backend
..
core: Stop overriding id of new permission in backend
Remove overriding of id of new permission
Vojtech Szocs has submitted this change and it was merged.
Change subject: gwt: Emulate random UUID generation in GWT
..
gwt: Emulate random UUID generation in GWT
Adds emulation of random UUID generation in GWT code that is tr
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: VM general refresh
..
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/41986
To unsubscribe, visit https://gerrit.ovirt.org/setti
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: storage main tab 500
..
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/42095
To unsubscribe, visit https://gerrit.ovirt.org/set
Vojtech Szocs has posted comments on this change.
Change subject: core: Stop overriding id of new permission in backend
..
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/41984
To unsubscribe, visit https
Vojtech Szocs has posted comments on this change.
Change subject: gwt: Emulate random UUID generation in GWT
..
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/41983
To unsubscribe, visit https://gerrit.o
Vojtech Szocs has posted comments on this change.
Change subject: core: Stop overriding id of new permission in backend
..
Patch Set 2: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/41984
To unsubscribe, visit https
Vojtech Szocs has submitted this change and it was merged.
Change subject: webadmin,userportal: ClientStorage improvements
..
webadmin,userportal: ClientStorage improvements
1. All key/value pairs persisted through ClientStorag
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: ClientStorage improvements
..
Patch Set 3:
Rebased, no changes since last patchset.
--
To view, visit https://gerrit.ovirt.org/41404
To un
Vojtech Szocs has submitted this change and it was merged.
Change subject: webadmin,userportal: Persistent client-side logging infra
..
webadmin,userportal: Persistent client-side logging infra
This patch adds following changes
Vojtech Szocs has posted comments on this change.
Change subject: frontend: fixed NPEs in debug mode
..
Patch Set 1:
I think it's worth to spend time and investigate why are we sending
PersistentBag instances to frontend in th
Vojtech Szocs has submitted this change and it was merged.
Change subject: webadmin: Null pointer dereferences in model classes
..
webadmin: Null pointer dereferences in model classes
Added null check when calling Configurator#
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: cluster guide me datacenter
..
Patch Set 1:
Sorry, I missed your conversation with Alex at
https://gerrit.ovirt.org/#/c/40782/ - please discard my com
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: cluster guide me datacenter
..
Patch Set 1:
Shmuel, with this patch applied, is your patch
https://gerrit.ovirt.org/#/c/40782/ still needed? (AFAIK, t
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: cluster guide me datacenter
..
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/41486
To unsubscribe, visit https://gerrit.ovirt.
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: ClientStorage improvements
..
Patch Set 2:
Tested instance creator role in UserPortal with Tomas, all seems to work well.
--
To view, visi
Vojtech Szocs has posted comments on this change.
Change subject: core, webadmin: Dismissible events
..
Patch Set 5: Code-Review+1
Frontend part looks good.
--
To view, visit https://gerrit.ovirt.org/41382
To unsubscribe, vis
Vojtech Szocs has posted comments on this change.
Change subject: core, webadmin: Dismissible events
..
Patch Set 3:
(1 comment)
https://gerrit.ovirt.org/#/c/41382/3/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovir
Vojtech Szocs has posted comments on this change.
Change subject: core, webadmin: Dismissible events
..
Patch Set 3: Code-Review+1
(4 comments)
Nice patch overall, some inline comments for your consideration.
https://gerrit.o
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: ClientStorage improvements
..
Patch Set 1:
Rerun-Hooks: all
--
To view, visit https://gerrit.ovirt.org/41404
To unsubscribe, visit https:/
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: ClientStorage improvements
..
Patch Set 1:
> http://jenkins.ovirt.org/job/ovirt-engine_master_find-bugs_gerrit/35833/ :
> The patch does no
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: Persistent client-side logging infra
..
Patch Set 14:
Rerun-Hooks: all
--
To view, visit https://gerrit.ovirt.org/25444
To unsubscribe, vi
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: Persistent client-side logging infra
..
Patch Set 14:
> http://jenkins.ovirt.org/job/ovirt-engine_master_find-bugs_gerrit/35832/ :
> The pa
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: Null pointer dereferences in model classes
..
Patch Set 2: Verified+1
--
To view, visit https://gerrit.ovirt.org/41405
To unsubscribe, visit https://g
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: Null pointer dereferences in model classes
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/41405
To unsubscribe, visit https://g
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: ClientStorage improvements
..
Patch Set 1:
@Tomas, please review my comments in UserPortal SystemModule.java - thanks!
--
To view, visit h
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: ClientStorage improvements
..
Patch Set 1:
(2 comments)
https://gerrit.ovirt.org/#/c/41404/1/frontend/webadmin/modules/userportal-gwtp/src/
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: ClientStorage improvements
..
Patch Set 1: Verified+1
Screenshot of Firefox / Dev Tools / local storage inspection:
http://i.imgur.com/MsGX
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: Persistent client-side logging infra
..
Patch Set 14:
> I'll write a patch to modify key format used to persist data in browser's
> local s
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: Persistent client-side logging infra
..
Patch Set 14:
Rebase, no changes since last patchset.
--
To view, visit https://gerrit.ovirt.org/2
Vojtech Szocs has uploaded a new change for review.
Change subject: webadmin,userportal: ClientStorage improvements
..
webadmin,userportal: ClientStorage improvements
1. All key/value pairs persisted through ClientStorage interf
Vojtech Szocs has uploaded a new change for review.
Change subject: webadmin: Null pointer dereferences in model classes
..
webadmin: Null pointer dereferences in model classes
Added null check when calling Configurator#getSynta
Vojtech Szocs has posted comments on this change.
Change subject: userportal, webadmin: automatically show header tooltip
underline
..
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/41298
To unsubscribe
Vojtech Szocs has posted comments on this change.
Change subject: userportal, webadmin: added visual indicator for tooltips
..
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/41296
To unsubscribe, visit h
Vojtech Szocs has posted comments on this change.
Change subject: userportal, webadmin: added visual indicator for tooltips
..
Patch Set 1:
Rerun-Hooks: all
--
To view, visit https://gerrit.ovirt.org/41296
To unsubscribe, vis
Vojtech Szocs has posted comments on this change.
Change subject: userportal, webadmin: fix broken helptag implementation
..
Patch Set 2: Code-Review+1
Looks good, it's basically +2 from me.
--
To view, visit https://gerrit.o
Vojtech Szocs has posted comments on this change.
Change subject: userportal, webadmin: removed unused images and css
..
Patch Set 3: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/40106
To unsubscribe, visit https:/
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: Optimize syntax checker handling
..
Patch Set 2:
Greg, thanks for your review!
--
To view, visit https://gerrit.ovirt.org/41284
To unsubsc
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: Optimize syntax checker handling
..
Patch Set 2: Verified+1
Verified on IE8 (Win2008R2 & Win7) - this patch reduces the delay between
hitti
Vojtech Szocs has uploaded a new change for review.
Change subject: webadmin,userportal: Optimize syntax checker handling
..
webadmin,userportal: Optimize syntax checker handling
1. access to (search) syntax checker instance is
Vojtech Szocs has submitted this change and it was merged.
Change subject: webadmin,userportal: Optimize syntax checker handling
..
webadmin,userportal: Optimize syntax checker handling
1. access to (search) syntax checker inst
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: Optimize syntax checker handling
..
Patch Set 1:
Thanks Tomas for review!
--
To view, visit https://gerrit.ovirt.org/41237
To unsubscribe,
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: Optimize syntax checker handling
..
Patch Set 1:
Thanks Greg for your review.
Adding some more UI maintainers, I'd like to have at least on
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: Optimize syntax checker handling
..
Patch Set 1: Verified+1
Did some additional sanity tests for both WebAdmin & UserPortal in Firefox,
ens
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: Optimize syntax checker handling
..
Patch Set 1:
Verified on IE8 together with https://gerrit.ovirt.org/#/c/41228/
This patch reduces the d
Vojtech Szocs has posted comments on this change.
Change subject: userportal, webadmin: fix IE8 rendering / jquery failure
..
Patch Set 1: Verified+1
Verified as part of verifying master patch https://gerrit.ovirt.org/#/c/41237
Vojtech Szocs has uploaded a new change for review.
Change subject: webadmin,userportal: Optimize syntax checker handling
..
webadmin,userportal: Optimize syntax checker handling
1. access to (search) syntax checker instance is
Vojtech Szocs has posted comments on this change.
Change subject: userportal, webadmin: fix IE8 rendering / jquery failure
..
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/41228
To unsubscribe, visit ht
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: UI plugins - control tab priority
..
Patch Set 4: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/41032
To unsubscribe, visit https://gerrit.
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: UI plugin API - setSearchString
..
Patch Set 4: Code-Review+1
All looks good, I'd like Alex to review this too, just in case.
--
To view, visit https
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: UI plugin revealPlace API - rename parameter
..
Patch Set 4: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/41031
To unsubscribe, visit http
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: UI plugin API - revealPlace
..
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/41029
To unsubscribe, visit https://gerrit.ovirt.
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: Persistent client-side logging infra
..
Patch Set 12:
To avoid clashes between WebAdmin vs. UserPortal instances running on the same
browse
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: Persistent client-side logging infra
..
Patch Set 12: Verified+1
All changes done and tested, patchset 12 is the one to review.
Verified in
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: Persistent client-side logging infra
..
Patch Set 11:
Tomas, thanks for your review!
I'll submit a final version of the patchset soon, sorr
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: Update host general layout.
..
Patch Set 2: Code-Review+1
All looks good!
--
To view, visit https://gerrit.ovirt.org/40886
To unsubscribe, visit http
Vojtech Szocs has submitted this change and it was merged.
Change subject: webadmin: UI plugins - control tab priority
..
webadmin: UI plugins - control tab priority
When adding custom main or sub tab, UI plugins can now specif
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: UI plugins - control tab priority
..
Patch Set 4:
Verified using test UI plugin attached to BZ https://bugzilla.redhat.com/1214624
--
To view, visit
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: UI plugins - control tab priority
..
Patch Set 4: Verified+1
Last patchset with some small tweaks.
--
To view, visit https://gerrit.ovirt.org/40510
T
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: UI plugins - control tab priority
..
Patch Set 1:
One more patchset coming up soon to improve the logic for comparing tab
priorities (represented as f
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: UI plugins - control tab priority
..
Patch Set 1:
@Alex, can you please review this one?
--
To view, visit https://gerrit.ovirt.org/40510
To unsubscr
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: UI plugins - control tab priority
..
Patch Set 1:
> tested and it is working great.
Thanks, Karnan. Let me quickly rebase and re-check before we can m
Vojtech Szocs has posted comments on this change.
Change subject: userportal, webadmin: Number field reporting wrong error
..
Patch Set 9: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/37244
To unsubscribe, visit ht
Vojtech Szocs has posted comments on this change.
Change subject: userportal, webadmin: Number field reporting wrong error
..
Patch Set 8:
(2 comments)
https://gerrit.ovirt.org/#/c/37244/8/frontend/webadmin/modules/gwt-common/
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: UI plugins - control tab priority
..
Patch Set 1:
Great, so this means it works? :-)
Have you tried different "priority" values for a custom main/sub
Vojtech Szocs has posted comments on this change.
Change subject: userportal, webadmin: show tooltips on cropped input boxes
..
Patch Set 3:
(1 comment)
https://gerrit.ovirt.org/#/c/39121/3/frontend/webadmin/modules/gwt-common
Vojtech Szocs has posted comments on this change.
Change subject: userportal, webadmin: show tooltips on cropped input boxes
..
Patch Set 3:
(2 comments)
In general looks OK, posted some comments, but I see that you will proba
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: fluent UiAction
..
Patch Set 47: Code-Review+2
Looks good, thanks for addressing all review comments.
--
To view, visit https://gerrit.ovirt.org/3684
Vojtech Szocs has posted comments on this change.
Change subject: userportal, webadmin: Number field reporting wrong error
..
Patch Set 8:
(3 comments)
https://gerrit.ovirt.org/#/c/37244/8/frontend/webadmin/modules/gwt-common/
Vojtech Szocs has posted comments on this change.
Change subject: userportal, webadmin: Number field reporting wrong error
..
Patch Set 7:
(1 comment)
https://gerrit.ovirt.org/#/c/37244/7/frontend/webadmin/modules/gwt-common/s
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: fluent UiAction
..
Patch Set 45: Code-Review+1
Giving +1 (see my inline comments for your consideration), Alex should take a
look as well.
--
To vie
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: fluent UiAction
..
Patch Set 45:
(10 comments)
https://gerrit.ovirt.org/#/c/36848/45//COMMIT_MSG
Commit Message:
Line 5: CommitDate: 2015-05-05 10:14
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: fluent UiAction
..
Patch Set 45:
Wow that's a lot of patchsets, going to look at the most recent one.
--
To view, visit https://gerrit.ovirt.org/3684
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: fluent UiAction
..
Patch Set 25:
(1 comment)
https://gerrit.ovirt.org/#/c/36848/25/frontend/webadmin/modules/uicommonweb/src/test/java/org/ovirt/engin
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: cluster guide me datacenter
..
Patch Set 3: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/37763
To unsubscribe, visit https://gerrit.ovirt.
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: cluster guide me datacenter
..
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/37763/2/frontend/webadmin/modules/webadmin/src/main/java/org/ovir
Vojtech Szocs has uploaded a new change for review.
Change subject: webadmin: UI plugins - control tab priority
..
webadmin: UI plugins - control tab priority
When adding custom main or sub tab, UI plugins can now specify
the pr
Vojtech Szocs has submitted this change and it was merged.
Change subject: webadmin,userportal: Disable GWTP version checking
..
webadmin,userportal: Disable GWTP version checking
Disabled GWTP VersionInspectorLinker functional
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: Disable GWTP version checking
..
Patch Set 1:
> Yeah, it is sad that we won't be able to upgrade to newer GWT/GWT-P due to
> the de-RPC res
Vojtech Szocs has submitted this change and it was merged.
Change subject: webadmin: UI plugin revealPlace API - rename parameter
..
webadmin: UI plugin revealPlace API - rename parameter
Renamed placeToken to historyToken for
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: UI plugin revealPlace API - rename parameter
..
Patch Set 1:
Thanks Alex for quick review.
--
To view, visit https://gerrit.ovirt.org/40494
To unsubs
Vojtech Szocs has uploaded a new change for review.
Change subject: webadmin,userportal: Disable GWTP version checking
..
webadmin,userportal: Disable GWTP version checking
Disabled GWTP VersionInspectorLinker functionality in o
Vojtech Szocs has posted comments on this change.
Change subject: webadmin,userportal: Disable GWTP version checking
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/40502
To unsubscribe, visit https://ger
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: UI plugin revealPlace API - rename parameter
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/40494
To unsubscribe, visit https:/
Vojtech Szocs has uploaded a new change for review.
Change subject: webadmin: UI plugin revealPlace API - rename parameter
..
webadmin: UI plugin revealPlace API - rename parameter
Renamed placeToken to historyToken for consiste
Vojtech Szocs has submitted this change and it was merged.
Change subject: webadmin: UI plugin API - setSearchString
..
webadmin: UI plugin API - setSearchString
This patch adds new UI plugin API function ('setSearchString')
th
Vojtech Szocs has submitted this change and it was merged.
Change subject: webadmin: UI plugin API - revealPlace
..
webadmin: UI plugin API - revealPlace
This patch adds new UI plugin API function ('revealPlace')
that can be us
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: UI plugin API - setSearchString
..
Patch Set 2: Verified+1
Verified using test UI plugin attached to BZ https://bugzilla.redhat.com/1081028
No impact
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: UI plugin API - setSearchString
..
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/40405/2/frontend/webadmin/modules/uicommonweb/src/main/java/o
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: UI plugin API - setSearchString
..
Patch Set 1:
Testing reveals that current implementation is quirky, need to update this
patch.
--
To view, visit
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: UI plugin API - revealPlace
..
Patch Set 1:
> Would it make sense to add another function to list the available places for
> the plugin. To allow for
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: UI plugin API - revealPlace
..
Patch Set 1: Verified+1
Verified using test UI plugin attached to BZ https://bugzilla.redhat.com/1168334
--
To view, v
Vojtech Szocs has uploaded a new change for review.
Change subject: webadmin: UI plugin API - setSearchString
..
webadmin: UI plugin API - setSearchString
This patch adds new UI plugin API function ('setSearchString')
that can b
Vojtech Szocs has uploaded a new change for review.
Change subject: webadmin: UI plugin API - revealPlace
..
webadmin: UI plugin API - revealPlace
This patch adds new UI plugin API function ('revealPlace')
that can be used to re
Vojtech Szocs has posted comments on this change.
Change subject: webadmin: Replace usage of *PmProxyPreferences() with
*FenceProxySources()
..
Patch Set 2: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/39761
To un
Vojtech Szocs has submitted this change and it was merged.
Change subject: frontend: Add infrastructure for Console Foreign Menu
..
frontend: Add infrastructure for Console Foreign Menu
Added necessary hooks to implement output
Vojtech Szocs has posted comments on this change.
Change subject: frontend: Add infrastructure for Console Foreign Menu
..
Patch Set 9: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/38986
To unsubscribe, visit https
1 - 100 of 2887 matches
Mail list logo
