Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Images alignment do not behave consistently in the UI on IE 11 #1184

Closed
ajsampang67 opened this issue Mar 18, 2019 · 9 comments
Closed

Images alignment do not behave consistently in the UI on IE 11 #1184

ajsampang67 opened this issue Mar 18, 2019 · 9 comments

Comments

@ajsampang67
Copy link

Do you want to request a feature or report a bug?
Bug

What is the current behavior?
When adding an image, an inital toolbar will pop up for alignment. Selecting one of these alignments will incorrectly highlight a blue box filling the editor area, and will not highlight which alignment is selected in the toolbar.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.

  1. Add a picture to the beginning of the test content (same line)
  2. When image is added, the alignment text box will appear. Click center
    **Important if clicking elsewhere first, the issue will not reproduce

What is the expected behavior?
The image would center, with the floating menu UI showing that the selected content is centered and no blue box extending all the way across the content section

Which versions of alloy-editor, and which browser / OS are affected by this issue? Did this work in previous versions?
IE11 only, all versions of alloy-editor. Unknown if it worked in previous versions.

@julien
Copy link
Contributor

julien commented Mar 19, 2019

Hi @ajsampang67,

Thanks for reporting this.

I tried reproducing the issue you described but I'm not really sure if this exactly what you mean, please have a look at the following .gif image

ae_demo_ie_portal

I'm inserting an image and hitting the "enter" key before trying to align the image, which will move the cursor "outside" of the image so I'm not sure if this should be considered as a bug.

@ajsampang67
Copy link
Author

ajsampang67 commented Mar 19, 2019

Hi @julien ,
No need to hit enter; as in the steps, once the image is added, that toolbar appears above the image (as in your gif). At this point, you should click "center" on that toolbar. Just as well in the note, you should not click anywhere else (or hit enter) before selecting the "center".

test

Here, I center it twice. First demonstrates the toolbar issue, where the toolbar will stay back in the original position and the center button is not highlighted. Second demonstrates the center outline, which should not be stretching across the editor, but only outlining the image (as shown in firefox/chrome)

Here is the expected for quick reference:
expected

@ajsampang67 ajsampang67 changed the title Images alignement do not behave consistently in the UI on IE 11 Images alignment do not behave consistently in the UI on IE 11 Mar 19, 2019
@julien
Copy link
Contributor

julien commented Mar 20, 2019

Hi @ajsampang67,
Thanks for the reply.

Concerning the image alignment button issue I sent PR #1185 which fixes the problem.
Concerning the "center outline" I don't think this is an issue.

@ajsampang67
Copy link
Author

Wonderful, thanks for the quick fix!

@julien good to close then?

@julien
Copy link
Contributor

julien commented Mar 20, 2019

@ajsampang67 no problem.
Concerning closing the issue, I'd rather do it once it's merged if you don't mind.
Thanks

@julien julien closed this as completed Mar 21, 2019
@ajsampang67
Copy link
Author

Hi @julien,

I've tested with the updated version and issue is still reproducible:
beta3_reproduced

@julien
Copy link
Contributor

julien commented Mar 25, 2019

Hi @ajsampang67 ,

Thanks for the update,

I tried this too but on the alloy-editor demo as you can see in the following .gif

ae_2019_03_25_ie11

I'll look into this when I have the time and keep you updated, for issues related to liferay-portal it might be a better idea to create an issue on Jira ticket.

Thanks

@julien
Copy link
Contributor

julien commented Mar 28, 2019

Hi @ajsampang67,

Just wanted to tell you I also tested this in liferay-portal and as far as I can see it's working as expected, please see the attached .gif

ae_2019_03_28_ie11

@ajsampang67
Copy link
Author

Hi @julien,
Looking back at the original steps, in step 2 there is an asterisked note saying "If clicking elsewhere first, the issue will not reproduce". It looks like in your reproduction, you're clicking near the bottom right of your screen, resulting in the initial menu disappearing. This will result in the issue not reproducing.
Here is my example: First I'll upload the image and click elsewhere, having the issue no longer reproducible. Second time, I will directly click the toolbar as soon as it shows up, reproducing the issue:

1184

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants