Fixed a bug where the contextual menu was being incorrectly positioned when zoom in on the webpage
This commit is contained in:
parent
0eb15085e9
commit
dca4702b7b
2 changed files with 7 additions and 4 deletions
|
@ -251,8 +251,10 @@ module.exports = React.createClass({
|
||||||
onEditClicked: function(e) {
|
onEditClicked: function(e) {
|
||||||
var MessageContextMenu = sdk.getComponent('rooms.MessageContextMenu');
|
var MessageContextMenu = sdk.getComponent('rooms.MessageContextMenu');
|
||||||
var buttonRect = e.target.getBoundingClientRect()
|
var buttonRect = e.target.getBoundingClientRect()
|
||||||
var x = buttonRect.right;
|
|
||||||
var y = buttonRect.top + (e.target.height / 2);
|
// The window X and Y offsets are to adjust position when zoomed in to page
|
||||||
|
var x = buttonRect.right + window.pageXOffset;
|
||||||
|
var y = buttonRect.top + (e.target.height / 2) + window.pageYOffset;
|
||||||
var self = this;
|
var self = this;
|
||||||
ContextualMenu.createMenu(MessageContextMenu, {
|
ContextualMenu.createMenu(MessageContextMenu, {
|
||||||
mxEvent: this.props.mxEvent,
|
mxEvent: this.props.mxEvent,
|
||||||
|
|
|
@ -77,8 +77,9 @@ module.exports = React.createClass({
|
||||||
onBadgeClicked: function(e) {
|
onBadgeClicked: function(e) {
|
||||||
var Menu = sdk.getComponent('rooms.NotificationStateContextMenu');
|
var Menu = sdk.getComponent('rooms.NotificationStateContextMenu');
|
||||||
var elementRect = e.target.getBoundingClientRect();
|
var elementRect = e.target.getBoundingClientRect();
|
||||||
var x = elementRect.right;
|
// The window X and Y offsets are to adjust position when zoomed in to page
|
||||||
var y = elementRect.top + (elementRect.height / 2);
|
var x = elementRect.right + window.pageXOffset;
|
||||||
|
var y = elementRect.top + (elementRect.height / 2) + window.pageYOffset;
|
||||||
var self = this;
|
var self = this;
|
||||||
ContextualMenu.createMenu(Menu, {
|
ContextualMenu.createMenu(Menu, {
|
||||||
left: x,
|
left: x,
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue