Skip to content

feat(ui): add community feedback banner to magma login view#3359

Closed
clutester wants to merge 1 commit into
apache:masterfrom
clutester:feature/community-feedback-banner
Closed

feat(ui): add community feedback banner to magma login view#3359
clutester wants to merge 1 commit into
apache:masterfrom
clutester:feature/community-feedback-banner

Conversation

@clutester
Copy link
Copy Markdown
Contributor

Description

This PR introduces a lightweight community feedback banner to the Magma login screen (LoginView.vue).

Motivation and Context

To improve user engagement and streamline support, this banner provides direct, visible links to the official caldera@mitre.org feedback email and the Caldera Discord community. Placing this on the login screen ensures users know exactly where to find help, report issues, or engage with the community before they even authenticate into the platform.

Changes Made

  • Modified plugins/magma/src/views/LoginView.vue.
  • Appended a feedback-banner element to the Pug template, positioned unobtrusively below the login form.
  • Utilized native styling (has-text-grey, is-size-7, etc.) to ensure the banner blends seamlessly with the existing Caldera UI design language.
  • Added standard target="_blank" and rel="noopener noreferrer" attributes to external links for security.

How Has This Been Tested?

  • Confirmed the template compiles cleanly via npm run build and npm run dev.
  • Visually verified the layout, spacing, and link functionality on the local development server.
  • Verified the banner dynamically scales and does not break the vertical centering of the main login container.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 7, 2026

@clutester clutester closed this Apr 7, 2026
@clutester clutester deleted the feature/community-feedback-banner branch April 8, 2026 16:10
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.

1 participant