Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

headline, preview of search results, hide selection, deduplication button, quick filters #649

Merged
merged 4 commits into from
Nov 7, 2024

Conversation

m13v
Copy link
Contributor

@m13v m13v commented Nov 7, 2024


name: pull request
about: submit changes to the project
title: "[pr] "
labels: ''
assignees: ''


description

brief description of the changes in this pr.

related issue: #

type of change

  • bug fix
  • new feature
  • breaking change
  • documentation update

how to test

add a few steps to test the pr in the most time efficient way.

if relevant add screenshots or screen captures to prove that this PR works to save us time.

checklist

  • MOST IMPORTANT: this PR will require less than 30 min to review, merge, and release to production and not crash in the hand of thousands of users
  • i have read the CONTRIBUTING.md file
  • i have updated the documentation if necessary
  • my changes generate no new warnings
  • i have added tests that prove my fix is effective or that my feature works

additional notes

any other relevant information about the pr.

Copy link

vercel bot commented Nov 7, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
screenpipe ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 7, 2024 5:07am

@@ -39,7 +39,7 @@ export function ContextUsageIndicator({
/>
</svg>
{percentage > 90 && (
<AlertTriangle className="w-5 h-5 absolute top-1/2 left-1/2 transform -translate-x-1/2 -translate-y-1/2 text-black" />
<AlertTriangle className="w-5 h-5 absolute top-1/2 left-1/2 transform -translate-x-1/2 -translate-y-1/2 text-red-500" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

fine for now but i'd prefer to keep our design black and white until we have a defined color palette

@louis030195 louis030195 merged commit 65f0eaf into mediar-ai:main Nov 7, 2024
4 of 7 checks passed
@louis030195
Copy link
Collaborator

@m13v

Screenshot 2024-11-07 at 9 06 31 AM

alignment, spacing, and size are wrong

we spread the attention of user on many things instead of an self-explanatory UX with only the most important & used things

image

image

image

image

image
Screenshot 2024-11-07 at 9 10 30 AM

Screenshot 2024-11-07 at 9 10 39 AM

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