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

Add New Nested Layout for Editor Page #1079

Closed
John-Paul-Larkin opened this issue Oct 7, 2024 · 12 comments · Fixed by #1163
Closed

Add New Nested Layout for Editor Page #1079

John-Paul-Larkin opened this issue Oct 7, 2024 · 12 comments · Fixed by #1163

Comments

@John-Paul-Larkin
Copy link
Member

John-Paul-Larkin commented Oct 7, 2024

Context

As part of "Meta issue : Editor #1022 we need to add a new nested layout for the editor page.
This new layout will be a sticky nav and contain navigation and other things relating specifically to the editor.

  • Remove "Your posts" and "+ New post" from the navigation bar, as they are not necessary on the editor page.
  • Remove "Save status indicator, and the "Publish" button from the editor(as they will now be on the nav)

Add a sticky navigation bar that includes elements and actions relevant to the editor, such as:

  • Codu logo (which remains as a home link).

  • Notifications.

  • Dark mode toggle.

  • User icon.

  • Move the "Publish" button from the editor to the sticky navigation.

  • Add a status indicator to reflect the save status of the editor:
    Note - Check out how this should be implemented in Nialls Figma image in the comment below.

- The status indicator should disappear while the user is typing.
- After a short delay once typing stops, a "Saving..." message should appear, possibly accompanied by a loading spinner.
- Once the content is successfully saved, the message should change to "Saved".

Take a look at how medium.com implements their editor nav to get a sense of the user experience and design we're aiming for.

Screenshots

Current nav

Additional info

The alpha editor can be found at :

app>(app)>alpha>new

Any questions feel free to ask.

@NiallJoeMaher
Copy link
Contributor

Example of what the nav should look like on editor screen

@eliezerjoel
Copy link

Heelo
I hope I can start working on this
no one is assigned yet

@John-Paul-Larkin
Copy link
Member Author

@eliezerjoel
I assigned you.

@NiallJoeMaher
Copy link
Contributor

Unassigned because there was no activity. If It's WIP, let me know.

@eliezerjoel
Copy link

Apologies
We had cut-offs ofpower and internet problems in my area over the past few days
If it has not been taken, I'm still eager to work on this issue

@John-Paul-Larkin
Copy link
Member Author

@eliezerjoel
Have you made any progress on this, or should I un-assign you and make it available to someone else?

@sairiteshdomakuntla
Copy link

hello @John-Paul-Larkin , i think i can do this, can you assign this to me?

@John-Paul-Larkin
Copy link
Member Author

Thanks @sairiteshdomakuntla
I assigned you

@sairiteshdomakuntla
Copy link

@John-Paul-Larkin , how do i need to connect my own backend, or you will provide the DB_URL?

@John-Paul-Larkin
Copy link
Member Author

HI @sairiteshdomakuntla,
Have you made progress, or can I reassign this issue?

@sairiteshdomakuntla
Copy link

Hello @John-Paul-Larkin , sorry I was facing many issues in setting up, you can reassign the issue to some other person

@NiallJoeMaher
Copy link
Contributor

I'll take this one @John-Paul-Larkin

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

Successfully merging a pull request may close this issue.

4 participants