Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

refactor: increase workspace and template avatar size#17200

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
BrunoQuaresma merged 2 commits intomainfrombq/avatars
Apr 2, 2025

Conversation

BrunoQuaresma
Copy link
Collaborator

Before
Screenshot 2025-04-01 at 14 46 10
Screenshot 2025-04-01 at 14 45 55

After
Screenshot 2025-04-01 at 14 46 18
Screenshot 2025-04-01 at 14 46 02

@BrunoQuaresmaBrunoQuaresma self-assigned thisApr 1, 2025
@BrunoQuaresmaBrunoQuaresma requested review froma team andKira-Pilot and removed request fora team andKira-PilotApril 1, 2025 17:48
Copy link
Contributor

@CopilotCopilotAI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors UI components to increase the avatar size for both workspaces and templates while also updating some typography settings in the Tailwind configuration.

  • Updated Tailwind configuration for font sizes
  • Changed the label from “User:” to “Owner:” in the workspaces table
  • Updated Avatar components in both workspaces and templates pages to use a larger size, along with layout adjustments in related components

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
FileDescription
site/tailwind.config.jsUpdated xs font size configuration to adjust line-height
site/src/pages/WorkspacesPage/WorkspacesTable.tsxRenamed label and set the avatar size to "lg"
site/src/pages/TemplatesPage/TemplatesPageView.tsxSet the avatar size to "lg" for template rows
site/src/components/Avatar/AvatarDataSkeleton.tsxReplaced Stack layout with div-based flex layout and updated Skeleton size
site/src/components/Avatar/AvatarData.tsxRefactored layout using div elements and Tailwind classes for text styling
Comments suppressed due to low confidence (1)

site/tailwind.config.js:18

  • The change in the xs font size from a line-height of '1.125rem' to '1rem' could affect overall text spacing in the UI. Please ensure this adjustment is intentional and aligns with the design requirements.
xs: ["0.75rem", "1rem"],

@BrunoQuaresmaBrunoQuaresma requested review fromaslilac,bpmct andmatifali and removed request fora teamApril 1, 2025 17:48
@BrunoQuaresmaBrunoQuaresma merged commit4604f19 intomainApr 2, 2025
30 checks passed
@BrunoQuaresmaBrunoQuaresma deleted the bq/avatars branchApril 2, 2025 00:29
@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsApr 2, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

Copilot code reviewCopilotCopilot left review comments

@aslilacaslilacaslilac approved these changes

@matifalimatifaliAwaiting requested review from matifali

@bpmctbpmctAwaiting requested review from bpmct

Assignees

@BrunoQuaresmaBrunoQuaresma

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@BrunoQuaresma@aslilac

[8]ページ先頭

©2009-2025 Movatter.jp