Refactor e melhorias na UI, estado global e experiência de uso #7
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… for scene management feat: update routing to include Productions page and adjust Studio path feat: implement reusable Button component with variants and sizes feat: create Card component for displaying production information feat: implement Column component for drag-and-drop functionality feat: add Header component with dynamic scene information feat: create HeaderScene component for scene filtering and creation feat: implement Input component with variants and error handling feat: create Layout component to structure the application feat: implement Scene component for displaying individual scenes feat: create SceneModal for scene details and editing feat: add production context for managing production state feat: define production types and actions for context management feat: implement scenes context for managing scenes state and actions feat: create utility functions for safe API fetching feat: implement Productions page to display and navigate to productions
…cross components - Updated import statements to use consistent spacing and formatting in various components including Button, Card, Header, Input, Layout, Scene, Sidebar, and Production contexts. - Enhanced readability by restructuring interface definitions and function parameters with consistent formatting. - Improved error handling and logging in safeFetch utility. - Refactored scene and production context logic for better clarity and maintainability. - Adjusted component return statements to use consistent formatting. - Updated Vite configuration for improved readability and consistency.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📝 Descrição do PR
Este PR traz uma série de melhorias estruturais e funcionais para o projeto, com foco em performance, manutenção de código e experiência do usuário.
✅ Principais melhorias realizadas:
Correção de atualização de estado fora do ciclo de renderização:
Evitei o erro "Cannot update a component while rendering a different component" movendo a lógica de seleção de produção para dentro de um
useEffect.Melhoria na estrutura de pastas (case-sensitive):
Ajustei inconsistências no case das pastas e arquivos (
Buttonvsbutton), seguindo boas práticas de organização e evitando problemas em ambientes case-sensitive.Adição de Skeleton Loaders (UI Feedback):
Implementei um Skeleton para melhorar a percepção de performance e feedback ao usuário durante carregamentos de dados.
Pequenos ajustes de performance e renderização:
Melhorias em pequenos pontos de render, como evitar execuções desnecessárias dentro do ciclo de renderização.
Melhorias gerais de código:
Pequenas refatorações visando melhor legibilidade, coesão e manutenção futura.
🧪 Como testar:
pnpm run dev);Por conta da mudança de case nas pastas (
button → Button), pode ser necessário para quem está no Mac/Windows dar umgit pull --forceou deletar localmente o antigo diretório antes de atualizar.