Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

Redesign contextual helper #2271

Merged
merged 7 commits into from
Jul 28, 2022
Merged

Redesign contextual helper #2271

merged 7 commits into from
Jul 28, 2022

Conversation

AlexAndBear
Copy link

@AlexAndBear AlexAndBear commented Jul 28, 2022

Description

Change: Redesign contextual helper

We've redesigned the contextual helper, which accepts now a title property and is able to display a description list.

Related Issue

Motivation and Context

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Screenshots (if appropriate):

image

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation added/updated

Open tasks:

  • ...

@update-docs
Copy link

update-docs bot commented Jul 28, 2022

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@AlexAndBear AlexAndBear requested a review from kulmann July 28, 2022 15:43
@AlexAndBear AlexAndBear requested a review from kulmann July 28, 2022 16:22
align-items: center;
}
.info-title {
font-size: 1.125rem;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we have a font size theming variable for this?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unfortunately no, but we use the same size in the OcButton.vue component

@sonarcloud
Copy link

sonarcloud bot commented Jul 28, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

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

Successfully merging this pull request may close these issues.

2 participants