Vojtech Szocs has submitted this change and it was merged. Change subject: userportal, webadmin: workaround for Firefox favicon bug ......................................................................
userportal, webadmin: workaround for Firefox favicon bug Workaround for Firefox bug https://bugzilla.mozilla.org/show_bug.cgi?id=519028, 'change location.hash leads to location bar icon (favicon) disappearance.' The symptom is that the favicon disappears in Firefox on the initial hashchange when logging in. The workaround is to simply detach and re-attach the favicon link element on hashchange by overriding ApplicationPlaceManager.onValueChange, and only in Firefox by checking the user-agent. Change-Id: Ia4b0d8a0176eb9299aec01ae6772297409ef3108 Bug-Url: https://bugzilla.redhat.com/999436 Signed-off-by: Greg Sheremeta <[email protected]> --- M backend/manager/modules/branding/src/main/resources/META-INF/tags/obrand/favicon.tag M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/place/ApplicationPlaceManager.java M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/place/UserPortalPlaceManager.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/place/WebAdminPlaceManager.java 4 files changed, 43 insertions(+), 7 deletions(-) Approvals: Vojtech Szocs: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20798 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia4b0d8a0176eb9299aec01ae6772297409ef3108 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Sheremeta <[email protected]> Gerrit-Reviewer: Alexander Wels <[email protected]> Gerrit-Reviewer: Einav Cohen <[email protected]> Gerrit-Reviewer: Greg Sheremeta <[email protected]> Gerrit-Reviewer: Vojtech Szocs <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
