Skip to content

Fix: Project Dataset & Notebook bugs#119

Open
KobenjiSan wants to merge 6 commits intodevelopfrom
fix/project-csv-display
Open

Fix: Project Dataset & Notebook bugs#119
KobenjiSan wants to merge 6 commits intodevelopfrom
fix/project-csv-display

Conversation

@KobenjiSan
Copy link
Collaborator

Includes bug fixes and UI refinements for the Notebooks and Datasets features.

Fixes:

  • Resolved issue where notebooks were not forking with projects
  • Fixed unreliable CSV loading behavior in the Datasets tab

UI Improvements:

  • Standardized Notebooks and Datasets tabs to utilize global styles
  • Improves button hierarchy and UI/UX patterns for clarity and increased user confidence

Backend
- Fixed project forking to properly clone notebooks and versions

Frontend
- Refactored notebook and targeted project UI to use global design tokens
- Improved button hierarchy, hover states, and visual consistency
Issue:
- currentDirectory was being derived from the route inconsistently
Fix:
- Removed string slicing that caused inconsistent routing
- Added normalized file routing logic

UI:
- Tweaked Datasets tab UI
- Improved button hierarchy
- Header shows Current folder / path clearly
- Clarified navigation by replacing the confusing “…” row with a clear back/close behavior
- Added preview context so it is more obvious what file is open
Copy link
Member

@cengique cengique left a comment

Choose a reason for hiding this comment

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

@KobenjiSan
There are some "unapproved" blue buttons under the CSV viewer and the visualization section.
Also, I tested forking and did not get the notebooks. Is that tested?

- csv preview now uses button styles that match entire system
- redesigned readme area with animations and clearer border structure
- rearranged buttons based on heiarchy and usecase
- moved project owner specific buttons to top of project for clear separation
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