Skip to content

Conversation

@DenysVuika
Copy link
Contributor

@DenysVuika DenysVuika commented Aug 20, 2024

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (check one with "x")

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation
  • Other... Please describe:

What is the current behaviour? (You can also link to an open issue here)

What is the new behaviour?

  • Cleanup the CSS workarounds from the Attach File Widget (APS)
  • Introduce new adf-alfresco-icon component (Alfresco logo)
  • Remove duplicated SVG images for Alfresco logo (Process Cloud library now does not require any assets copy)
  • fix icon position for Attach Widget
  • add "underline" effect on hover, to show the label is clickable
Screenshot 2024-08-20 at 1 16 27 PM Screenshot 2024-08-20 at 12 34 14 PM Screenshot 2024-08-20 at 12 52 17 PM

Does this PR introduce a breaking change? (check one with "x")

  • Yes
  • No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...

Other information:
https://hyland.atlassian.net/browse/ACS-8630

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
B Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@DenysVuika DenysVuika merged commit 8d7ea71 into develop Aug 21, 2024
@DenysVuika DenysVuika deleted the mat-cleanup-attach-file-widget branch August 21, 2024 12:00
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

Successfully merging this pull request may close these issues.

6 participants