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

MCP process list is empty (Chrome) or severely truncated (Firefox, Edge) #2057

Closed
3 tasks done
badvision opened this issue Sep 26, 2019 · 2 comments
Closed
3 tasks done
Milestone

Comments

@badvision
Copy link
Contributor

Required Information

  • AEM Version, including Service Packs, Cumulative Fix Packs, etc: Any
  • ACS AEM Commons Version: Any
  • Reproducible on Latest? yes

Issue caused by browser changes

Looks like every browser vendor got together and decided life was too simple and it was time to change default CSS behavior in every major browser this week. The result is that the mcp-workspace div gets an incorrect default height. In Chrome that is 0 pixels, making the table disappear. In Firefox and others, the table appears but is only 50px or so tall so the contents appear truncated.

How to hot-fix

In the short term, you can use Sarafi and all the functionality still appears to work. If you want to hot-fix your environment open the CSS file in crx de at /apps/acs-commons/components/utilities/manage-controlled-processes/clientlibs/css/mcp.css and add a rule to #mcp-workspace to set height:100%

@badvision
Copy link
Contributor Author

Example of issue in Chrome:
image

@badvision
Copy link
Contributor Author

Example of issue in Firefox:

image

@badvision badvision changed the title MCP process list is empty (Chrome) or severely truncated (Firefox, Safari, Edge) MCP process list is empty (Chrome) or severely truncated (Firefox, Edge) Sep 26, 2019
badvision pushed a commit that referenced this issue Sep 27, 2019
badvision added a commit that referenced this issue Sep 27, 2019
* #2058 - Addresses issues with firefox in MCP

* #2057 - Force height to 100% on outer div so it doesn't disappear in the new version of chrome.
@badvision badvision added this to the 4.3.4 milestone Oct 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant