Skip to content

Comments

QasDialog: mudanças gerais.#1432

Open
DouglasCalora wants to merge 27 commits intodevelopfrom
feature/dialog-changes
Open

QasDialog: mudanças gerais.#1432
DouglasCalora wants to merge 27 commits intodevelopfrom
feature/dialog-changes

Conversation

@DouglasCalora
Copy link
Collaborator

@DouglasCalora DouglasCalora commented Nov 18, 2025

closes #1346
closes #1360
closes #1121

Não publicado

BREAKING CHANGES

  • Componentes que usam dialog:
    • QasBoardGenerator -> prop confirmDialogProps.
    • QasGallery -> prop dialogProps.
    • QasSelectListDialog -> prop dialogProps.
    • QasSignatureUploader -> prop dialogProps.
    • QasTextTruncate -> prop dialogProps.
    • QasUploader -> prop dialogProps.
    • QasDrawer -> prop dialogProps (prop persistent saiu do dialogProps e agora é passado diretamente para o QasDrawer).
  • QasDrawer: adicionado propriedade persistent para não utilizar dentro de dialogProps.
  • QasDialog:
    • removido prop card em favor de suar as props title e description.
    • removido slot actions em favor de fazer os controles somente por ok, cancel e tertiary.
    • removido prop persistent, agora é feito sempre de forma automática.
    • removido prop actionsProps.
    • removido props useFullMaxWidth, maxWidth, minWidth em favor de utilizar a prop size.

Adicionado

  • QasDialog:
    • adicionado propriedade size com default sm.
    • adicionado propriedades title e description para ser usado no lugar da prop card.
    • adicionada propriedade disableCloseButton para desabilitar botão de fechar (x).
    • adicionada propriedade tertiary.
    • adicionada propriedade useAutoCloseOnActions (#1360)(#1121)
    • adicionada propriedade useCloseButton para remover botão de fechar (x).
    • adicionada propriedade useHtmlDescription.
  • QasDrawer: adicionado propriedade persistent para não utilizar dentro de dialogProps.
  • QasDialog:
    • removido prop card em favor de suar as props title e description.
    • removido slot actions em favor de fazer os controles somente por ok, cancel e tertiary.
    • removido prop persistent, agora é feito sempre de forma automática.
    • removido prop actionsProps.
    • removido props useFullMaxWidth, maxWidth, minWidth em favor de utilizar a prop size.

Modificado

  • QasDialog:
    • mudanças gerais do dialog (#1346).
    • removido uso de composable em arquivos separados para centralizar tudo no arquivo vue.
  • QasHeader: modificado propriedade badges para aceitar objeto quando tiver apenas 1 badge.
  • QasSignatureUploader: Modificado label e tamanho do dialog.
  • Normalização dos dialogs:
    • QasSignatureUploader.
    • QasTreeGenerator.
    • QasTextTruncate.
    • QasSelectListDialog.
    • QasFormView.
    • QasGallery -> PvGalleryCarouselDialog.
    • composables/use-delete.js
    • QasDrawer.

Removido

  • QasDialog:
    • removido prop card em favor de suar as props title e description.
    • removido slot actions em favor de fazer os controles somente por ok, cancel e tertiary.
    • removido prop persistent, agora é feito sempre de forma automática.
    • removido prop actionsProps.
    • removido props useFullMaxWidth, maxWidth, minWidth em favor de utilizar a prop size.

Versão do asteroid

  • v2 -> a partir da branch v2.
  • v3-beta.x -> a partir da branch develop.
  • v3-stable -> a partir da branch main.

Tipo de alteração

  • Adicionado | Added (novos componentes e/ou funcionalidades);
  • Modificado | Changed (alterações que podem ou não conter breaking changes);
  • Corrigido | Fixed (correção de bugs, typos, etc);
  • Removido | removed (remoção de algum componente e/ou funcionalidade).

O que foi alterado/adicionado

  • CSS
  • Componentes
  • Composables (v3)
  • Diretivas
  • Documentação
  • Helpers
  • Mixins
  • Paginas
  • Plugins
  • Testes
  • Outros

Este pull request introduz algum breaking change?

  • Sim
  • Não

Checklist

  • Foi discutida anteriormente com os times de Frontend e Design;
  • Foi testado manualmente no ambiente de desenvolvimento (/docs se v3 ou ui/dev se v2);
  • Foi constatado que esta modificação não gerou erros ou alertas no Console;
  • Foi verificado se o código segue os padrões de escrita e validado com o ESLint;
  • Foi escrito teste automatizado;
  • Foi atualizada e testada a documentação;
  • Foi atualizado o changelog seguindo o padrão "Keep a Changelog";
  • Fiz meu próprio code review antes de abrir este pull request.

refactor: clean up dialog examples by removing unnecessary properties and restructuring dialogProps

feat: add support for HTML descriptions in QasDialog

feat: introduce tertiary button functionality in QasDialog

fix: enhance QasDrawer component to support persistent prop and improve dialog integration

docs: update dialog documentation with new size options and example usage
…ity and consistency; remove unused QasFormDialog component
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Nov 18, 2025

Deploying asteroid with  Cloudflare Pages  Cloudflare Pages

Latest commit: 89e63b3
Status: ✅  Deploy successful!
Preview URL: https://4043f45f.asteroid-akr.pages.dev
Branch Preview URL: https://feature-dialog-changes.asteroid-akr.pages.dev

View logs

Comment on lines +27 to +36
const dialogProps = computed(() => ({
useForm: true,
title: 'Título do dialog',
ok: {
label: 'Fechar',
loading: isLoading.value
},
onCancel,
onOk
}))
Copy link
Collaborator

Choose a reason for hiding this comment

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

não usamos com este padrão, usamos o return {}

desc: Tamanho máximo do dialog.
type: String
description:
desc: Descrição do dialog, pode ser uma string ou um componente ou uma string com HTML (passando a prop useHtmlDescription).
Copy link
Collaborator

Choose a reason for hiding this comment

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

... pode ser uma string, um componente ou uma string...

actions:
desc: Slot para ações (botões por exemplo).
tertiary:
desc: Props para o botão terciário
Copy link
Collaborator

Choose a reason for hiding this comment

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

seria bom explicar que o botão terciário só vai ter quando ter essa prop.

Props para o botão terciário, onde só terá o botão caso passe essa prop


<template #description>
<div :style="signaturePadWidth">
<div>
Copy link
Collaborator

Choose a reason for hiding this comment

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

precisa da div?

- QasTextTruncate -> prop `dialogProps`.
- QasUploader -> prop `dialogProps`.
- QasDrawer -> prop `dialogProps` (prop `persistent` saiu do `dialogProps` e agora é passado diretamente para o QasDrawer).
- `QasDrawer`: adicionado propriedade `persistent` para não utilizar dentro de `dialogProps`.
Copy link
Contributor

Choose a reason for hiding this comment

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

isso seria realmente um breaking change do drawer? faz sentido estar aqui?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sim porque muda como usa.

Comment on lines +16 to +17

const dialogProps = {
Copy link
Contributor

Choose a reason for hiding this comment

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

// consts
assim esta ficando junto com as refs

return {
label: 'Cancelar',
outline: true,
'data-cy': 'dialog-cancel-btn',
Copy link
Contributor

Choose a reason for hiding this comment

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

qual a necessidade do data agr?? ja comecei os testes?

Comment on lines 50 to 52
persistent: false,
cancel: false,
ok: false,
Copy link
Contributor

Choose a reason for hiding this comment

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

ja nao tem cancel e nem ok, pela regra comentada ate na doc, o persistent ja seria false, nao??

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sim, estava atoa.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants