Skip to content

Fixes #12- Enhanced the User Interface of Project - #16

Merged
0zXD merged 3 commits into
0zXD:hacktober/new-additionsfrom
vaibhav-katyal:fix-ui-ehancement
Oct 12, 2025
Merged

Fixes #12- Enhanced the User Interface of Project#16
0zXD merged 3 commits into
0zXD:hacktober/new-additionsfrom
vaibhav-katyal:fix-ui-ehancement

Conversation

@vaibhav-katyal

@vaibhav-katyal vaibhav-katyal commented Oct 9, 2025

Copy link
Copy Markdown

Closes #12

PR Description

This pull request focuses on enhancing the User Interface (UI) of the Memory Map.

Previous State

  • The UI was misaligned, unappealing, and not responsive.
  • Layout issues affected the overall user experience on different devices.

Enhancements Made

  • Redesigned the UI to be clean, modern, and visually appealing.
  • Improved alignment and spacing for better readability.
  • Made the interface fully responsive across various screen sizes.

Result

A polished, user-friendly, and responsive Memory Map interface that provides a better visual experience while keeping all the functionalities and sections intact.

-->

Screenshot 2025-10-09 073243 Screenshot 2025-10-09 073257 Screenshot 2025-10-09 073312 Screenshot 2025-10-09 073334

@0zXD

0zXD commented Oct 9, 2025

Copy link
Copy Markdown
Owner

The UI looks amazing, good work.
But i'm confused about one thing, why did you delete almost all the functionality of the site?

Here are some examples:

  1. The site, can't load existing memories, because you deleted that entire function:
image
  1. Cannot save any image to the database, because you deleted the POST function as well:
image
  1. Can't delete any memory because you deleted that function as well:
image

The frontend looks amazing. But in the back-end the entire functionality of the site is deleted.


And the issue you addressed about the form not being visible, it's because of a change you made in the showMemoryForm logic. Lemme show u.

Your code snippet:

showMemoryForm() {
    const form = document.getElementById('memoryForm');
    if (form) {
        form.classList.remove('hidden');
        form.classList.add('show-fixed');
        setTimeout(() => {
            const titleInput = document.getElementById('memoryTitle');
            if (titleInput) {
                titleInput.focus();
            }
        }, 100);
    }
}

Fixed code snipped:

showMemoryForm() {
    const form = document.getElementById('memoryForm');
    if (form) {
        form.classList.remove('hidden');
        form.style.transform = 'translateX(0)';
        form.style.opacity = '1';
        setTimeout(() => {
            const titleInput = document.getElementById('memoryTitle');
            if (titleInput) {
                titleInput.focus();
            }
        }, 100);
    }
}

The output should look something like this:

image

Kindly resolve the issue.

@vaibhav-katyal

Copy link
Copy Markdown
Author

Oh, sorry about that — I shouldn’t have interfered with the JS file.
I wasn’t able to add memory or test it properly at that time; otherwise, I would’ve fixed it in place.
No worries though, I’ll fix it right away.

@0zXD

0zXD commented Oct 9, 2025

Copy link
Copy Markdown
Owner

It'll be better if you could see the site's working first, then you'll be able to relate more to it.
If you needed any help to get the site to run, make sure to reach out.

@vaibhav-katyal

Copy link
Copy Markdown
Author

Can you please check once now, is every functionality working ?

@0zXD

0zXD commented Oct 9, 2025

Copy link
Copy Markdown
Owner

Perfect, that sure is a job well done.
Just want one more improvements:


The Memory Form touches the bottom of the computer screen, can you make it look better?

image

The PR will be merged as soon as this requirement is satisfied.

@vaibhav-katyal

Copy link
Copy Markdown
Author

I have fixed the the memory form touching bottom issue.
image

@vaibhav-katyal

Copy link
Copy Markdown
Author

Please merge the PR and add hacktoberfest-accepted label to the PR as currently your repository is not officially participating in hacktoberfest as shown on hacktoberfest.com

image

@0zXD
0zXD changed the base branch from master to hacktober/new-additions October 12, 2025 03:27
@0zXD
0zXD merged commit 1f01e7d into 0zXD:hacktober/new-additions Oct 12, 2025
@0zXD 0zXD added the hacktoberfest-accepted This PR is accepted under the hacktoberfest challenge. label Oct 12, 2025
@0zXD

0zXD commented Oct 12, 2025

Copy link
Copy Markdown
Owner

Sure, I'll make sure to implement the necessary changes.
Thank you for your contribution @vaibhav-katyal : )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest-accepted This PR is accepted under the hacktoberfest challenge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ENHANCEMENT] - Improve the User Interface (UI) for Memory Map

2 participants