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

feat: diff-view #1246

Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

Conversation

Toddyclipsgg
Copy link

@Toddyclipsgg Toddyclipsgg commented Feb 2, 2025

I made this resource due to its lack because I wanted to know which lines and sections LLM was changing. It is not perfect yet. If someone is willing to help me optimize and eliminate redundancies, we can make a great resource to be added to the project. @thecodacus @wonderwhy-er @coleam00 @mrsimpson @Stijnus @dustinwloring1988

Captura de tela 2025-02-01 202213
Captura de tela 2025-02-01 221111

@leex279 leex279 added the enhancement New feature or request label Feb 2, 2025
@Toddyclipsgg
Copy link
Author

I made some changes, I added a dropdown menu to choose the modified files and unify the blocks.

image

@thecodacus thecodacus added this to the 0.0.7 milestone Feb 3, 2025
@Toddyclipsgg
Copy link
Author

Tag it as "WIP. @thecodacus @leex279

@dustinwloring1988 dustinwloring1988 added the WIP Work In Progress label Feb 7, 2025
@Toddyclipsgg
Copy link
Author

Detailed Overview of Recent Changes:

00bed77 - refactor: update FileModifiedDropdown styling and toast notification
UI Improvements: The FileModifiedDropdown component now features updated styling to match the current design guidelines. This includes refined padding, adjusted colors, and improved hover effects, resulting in a more polished and cohesive appearance.

Enhanced Notifications: The toast notification used to confirm actions (like copying the file list) has been updated. This change makes the notification more prominent, ensuring users receive clear and consistent feedback when they perform file-related actions.

7e7fcec - fix: change markdown language detection to plaintext in DiffView
Bug Fix: Previously, markdown files were being interpreted in a way that could interfere with proper syntax highlighting or diff rendering. This commit changes the language-detection logic so that markdown files are treated as plaintext within the DiffView.

Consistent Rendering: Handling markdown as plaintext prevents any misinterpretation of markdown syntax during the diff process, leading to a more consistent and readable output.

50d7c43 - feat: refactor file history tracking and add NoChangesView component
File History Overhaul: The file history tracking mechanism has been refactored to improve its robustness and accuracy. Changes now ensure that any modifications to a file are tracked more reliably, which is crucial for proper diff computation.

NoChangesView Component: A new component—NoChangesView—has been introduced. This component clearly indicates to the user when no changes are detected between file versions, helping reduce confusion and providing a cleaner UI when files remain unmodified.

f0dff04 - feat: improve diff view with robust file change detection and error handling
Advanced Detection: This update enhances the diff view's ability to detect file changes, even in edge cases where differences could be subtle or complex. The improved detection mechanism minimizes false positives and ensures accurate visualization of changes.

Error Resilience: Additional error handling has been incorporated into the diff processing logic. In cases where diff computation might fail or encounter unexpected inputs, the system now responds more gracefully, providing useful feedback rather than a generic failure.

5fac72f - feat: add fullscreen and syntax highlighting to diff views
Fullscreen Mode: Users can now expand the diff view to fullscreen, offering an unobstructed view of the file differences. This feature is particularly useful for detailed code reviews.

Syntax Highlighting: Enhanced syntax highlighting within diff views makes code differences more visually distinct. This makes it easier to read and analyze changes, especially in large code blocks, by using color-coding that correlates with the underlying language syntax.

de1891f - feat: add side-by-side diff view and enhance diff component flexibility
Side-by-Side Comparison: In addition to the existing inline diff view, a new side-by-side layout has been implemented. This layout presents the original and modified code next to each other, allowing users to compare changes more intuitively.

Component Flexibility: The diff component has been made more flexible, supporting dynamic toggling between different diff views. This flexibility ensures that users have a choice in how they view the differences based on their current needs.

854bd2f - fix: revert package.json
Reversion: This commit reverts previous modifications made to the package.json file. The reversion was necessary to undo changes that had unintended negative impacts on dependency management or build configurations.

Stability: By reverting these changes, the package configuration is restored to a known stable state, helping maintain the integrity of the project’s build process.

af444e6 - chore: remove unused imports in DiffView and Workbench components
Code Cleanup: Unused imports in both the DiffView and Workbench components have been removed. This cleanup helps reduce unnecessary overhead, prevents potential naming conflicts, and ensures that the code is easier to maintain.

Optimized Bundle: Removing dead code also contributes to a slightly leaner bundle size, which may positively impact performance during compilation and runtime.

64a5f95 - feat: improve file changes tracking and diff view with enhanced history management
Enhanced Tracking: Further refinements have been made to the file changes tracking system. Improvements in history management allow the system to capture file modifications more comprehensively, ensuring that the diff view reflects accurate file evolution.

Unified Experience: The strengthened tracking mechanism not only benefits the diff view but also supports other UI elements that rely on correct file history data, leading to a more coherent user experience.

40e09cb - fix: remove unused import in CodeComparison component
Minor Fix: A redundant import in the CodeComparison component was removed. Although a small change, it contributes to overall code quality by eliminating potential warnings and keeping the module focused on its core functionality.

Better Readability: This change also aids maintainability, making the component easier to understand and navigate for future improvements or debugging.

I need someone to review the code and see if everything is correct @thecodacus @dustinwloring1988 @leex279. I still don't know why the changes appear several times in the same file even though it was modified once. I don't know if it's my fault or if Bolt opens the same file several times. The changes are being detected correctly regardless of how many times Bolt opens the file. There are still some improvements missing, but I don't know how to make them yet. The idea comes from a lack of seeing what Bolt was modifying.

Meu.Video-1.mp4

@Toddyclipsgg
Copy link
Author

@felvieira poderia me ajudar?

@felvieira
Copy link

@felvieira poderia me ajudar?

Do que vc precisaa?

@Toddyclipsgg
Copy link
Author

Eu fiz o recurso so que arrumei todos os erros na base do codigo mais ainda aparece erros no console e não consigo resolver ainda estou aprendendo tudo então é meio dificil. Se tu tiver alguma ideia de como resolver. @felvieira

@Toddyclipsgg
Copy link
Author

Feature ready. It's not perfect but it's a start.

@Ridoy302583
Copy link

Ridoy302583 commented Feb 10, 2025

""Create a Modern, responsive website for my portfolio of digital marketing work. It should have a contact form, a blog, a about page, a services section, and a homepage with an interesting hero section. Use blue, purple, and white as the main colors in a clean, expertly designed layout.""

Here is the Prompt. In here Continue Prompt not working. If possible working to solve this error
408277248-26e84e0b-69b4-43a3-99c7-25903e0dfd3e

@Toddyclipsgg
Copy link
Author

This has nothing to do with PR. @Ridoy302583

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request WIP Work In Progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants