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

HTML container component #1178

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

bschuhma
Copy link

Q                       A
Fixed Issues? New component - HTML Container component
Patch: Bug Fix?
Minor: New Feature?
Major: Breaking Change?
Tests Added + Pass? No - **** TO-DO -
Documentation Provided Yes (code comments and or markdown)
Any Dependency Changes?
License Apache License, Version 2.0

The HTML component addresses customers' needs/desires to write their own HTML, while implementing some element of formality by forcing them to use only files in the DAM. Instead of customers just free-handing code that hasn't undergone any form of peer review, hopefully this component will force them to review the HTML/CSS/JS and put it into the DAM.

The component allows for multiple CSS and JS files to support the one HTML file. The CSS files can be brought in inline or via reference, while the JS files can be brought in via async, defer, reference and inline.

@sonarcloud
Copy link

sonarcloud bot commented Sep 18, 2020

SonarCloud Quality Gate failed.

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 42 Code Smells

No Coverage information No Coverage information
82.2% 82.2% Duplication

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Sep 18, 2020

This pull request introduces 10 alerts when merging 0a99146 into 19c0a14 - view on LGTM.com

new alerts:

  • 6 for Implicit operand conversion
  • 3 for Variable not declared before use
  • 1 for Potential input resource leak

@vladbailescu vladbailescu added the move-to-contrib Contributions better suited to be added to https://github.com/adobe/aem-contrib-wcm-components label Jan 13, 2021
vladbailescu added a commit to adobe/aem-contrib-wcm-components that referenced this pull request Nov 9, 2021
HTML container component

* based on adobe/aem-core-wcm-components#1178

Co-authored-by: bschuhma <bschuhma@adobe.com>
Co-authored-by: Vishal Singh Arya <varya@adobe.com>
Co-authored-by: Abhinav Saraswat <abhinavsaraswat@Abhinavs-MacBook-Pro.local>
Co-authored-by: Vlad <vlad@bailescu.ro>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
move-to-contrib Contributions better suited to be added to https://github.com/adobe/aem-contrib-wcm-components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants