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

LPS-114149 Remove logic that was setting placeholder to BLANK on IE >=10 #92220

Conversation

liferay-continuous-integration
Copy link
Collaborator

Forwarded from: liferay-frontend#201 (Took 1 ci:forward attempt in 3 minutes)

@carloslancha
@liferay-frontend

Original pull request comment:
Removing logic that was setting placeholder to BLANK on IE >=10.

IE10 is no longer supported and behavior described in LPS-73502, where this fix was introduced in the first place, is no longer reproducible in IE11.

Steps to reproduce:

Using IE11

  • Go to blogs
  • Add new blog
  • Title and subtitle placeholders should appear
  • If no subtitle is introduced and blog is published, "subtitle" is not saved as this field value

✔️ ci:test:stable - 13 out of 13 jobs passed

✔️ ci:test:relevant - 34 out of 34 jobs passed in 4 hours 2 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: fc41ffa140788c094638ac1f5494781ee8770005

Copied in Private Modules Branch:

Branch Name: master-private
Branch GIT ID: a2989be147835c295b16fd5637140c9d4cd9de61

ci:test:stable - 13 out of 13 jobs PASSED
13 Successful Jobs:
ci:test:relevant - 34 out of 34 jobs PASSED
34 Successful Jobs:
For more details click here.

✔️ ci:test:sf - 1 out of 1 jobs passed in 4 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: fc41ffa140788c094638ac1f5494781ee8770005

Sender Branch:

Branch Name: LPS-114149-ie11-placeholder
Branch GIT ID: 0293463bdcf81b3e9caab5ca94ac2d77a48cfa3c

1 out of 1jobs PASSED
1 Successful Jobs:
For more details click here.

…=10. IE10 is no longer supported and behavior described in LPS-73502 is no longer reproducible in IE11.
@liferay-continuous-integration
Copy link
Collaborator Author

To conserve resources, the PR Tester does not automatically run for forwarded pull requests.

@brianchandotcom
Copy link
Owner

Merged. Thank you.
View total diff: 82af277...4316821

@liferay-continuous-integration liferay-continuous-integration deleted the ci-forward-LPS-114149-ie11-placeholder-pr-201-sender-carloslancha-ts-1596539161369 branch August 12, 2020 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants