Skip to content

refactor(ui): Clean frontend codebase & de-vibifi - #402

Open
siduck wants to merge 19 commits into
developfrom
refactor-ui
Open

refactor(ui): Clean frontend codebase & de-vibifi #402
siduck wants to merge 19 commits into
developfrom
refactor-ui

Conversation

@siduck

@siduck siduck commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

Simplify Sticky Toolbar component

  • To add a fade out blur effect on scroll the component adding unnecessary js hackery even tho we dont need this feature much
  • Negative margins for no reason ( just use padding at different breakpoints )
simplescreenrecorder-2026-08-18_15.38.00.mp4

Remove default props from frappe-ui components

  • Many components have default props for example <Button variant="subtle" size="md" and so on which is useless as they're default props we dont have to write them

@greptile-apps

greptile-apps Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Reviews (13): Last reviewed commit: "Merge branch 'develop' into refactor-ui" | Re-trigger Greptile

Comment thread admin/frontend/dashboard/src/components/common/StickyToolbar.vue Outdated
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.

1 participant