Skip to content

Update: address colors#2448

Merged
ItzNotABug merged 1 commit into
mainfrom
update-gradient-colors
Oct 5, 2025
Merged

Update: address colors#2448
ItzNotABug merged 1 commit into
mainfrom
update-gradient-colors

Conversation

@ItzNotABug

@ItzNotABug ItzNotABug commented Oct 5, 2025

Copy link
Copy Markdown
Contributor

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

Summary by CodeRabbit

  • Style
    • Refined column resizer and overlay visuals with a consistent border color for better clarity.
    • Adjusted gradients and fades to improve readability and contrast in both light and dark themes.
    • Brightened inner shimmer effect for more noticeable feedback during interactions.
    • Smoothed bottom fade transitions on sheets, enhancing depth and reducing visual noise.
    • Unified dark theme palette for overlays to improve visual consistency across views.
    • General polish to empty states for database tables to feel cleaner and more modern.

@ItzNotABug ItzNotABug self-assigned this Oct 5, 2025
@coderabbitai

coderabbitai Bot commented Oct 5, 2025

Copy link
Copy Markdown
Contributor

Walkthrough

Updates styling for empty states in database-related Svelte routes. Introduces a CSS variable (--columns-range-pink-border-color) and replaces hard-coded rgba values for borders, box-shadows, and overlays. Broadens a selector from &:not(:has(.columns-range-overlay.thinking)) to &:has(.columns-range-overlay). Adjusts light and dark theme gradients and opacity stops for bottom fades and inner shimmer effects. Adds a dark-theme declaration setting the new border-color variable on the outer container. No changes to exported or public entities.

Suggested reviewers

  • HarshMN2345

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title Check ❓ Inconclusive The title “Update: address colors” is too vague to convey the scope of the changes, which introduce a new CSS variable for column-range borders and adjust multiple gradient and overlay color values across themes. Please revise the title to clearly reflect the main changes, for example “Introduce CSS variable for columns-range border color and update gradients.”
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-gradient-colors

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cb49559 and 2ad94cb.

📒 Files selected for processing (2)
  • src/routes/(console)/project-[region]-[project]/databases/database-[database]/(suggestions)/empty.svelte (6 hunks)
  • src/routes/(console)/project-[region]-[project]/databases/database-[database]/table-[table]/layout/emptySheet.svelte (2 hunks)
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-09-30T07:41:06.679Z
Learnt from: ItzNotABug
PR: appwrite/console#2425
File: src/routes/(console)/project-[region]-[project]/databases/database-[database]/(suggestions)/empty.svelte:454-468
Timestamp: 2025-09-30T07:41:06.679Z
Learning: In `src/routes/(console)/project-[region]-[project]/databases/database-[database]/(suggestions)/empty.svelte`, the column suggestions API (console.suggestColumns) has a maximum limit of 7 columns returned, which aligns with the initial placeholder count of 7 in customColumns.

Applied to files:

  • src/routes/(console)/project-[region]-[project]/databases/database-[database]/(suggestions)/empty.svelte
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: build
  • GitHub Check: e2e

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ItzNotABug ItzNotABug merged commit fa1412d into main Oct 5, 2025
3 checks passed
@ItzNotABug ItzNotABug deleted the update-gradient-colors branch October 5, 2025 12: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.

2 participants