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

Moved Edit/Delete/Save buttons to the navigation bar #112

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

Conversation

reactorcoremeltdown
Copy link
Contributor

@reactorcoremeltdown reactorcoremeltdown commented Mar 27, 2023

Summary

Making editing buttons accessible when scrolling the contents. Also providing more space for filename + file path.

Details

I use wikmd a lot, also on the go, and I have noticed that on some long pages I have to scroll to top or to bottom to access Edit/Save buttons. This is somewhat inconvenient as I am losing the context as a result. This PR makes use of the fixed to top navigation bar and puts all important buttons there, making it possible to edit and save pages in the middle.

As usual, if the change does not fit into design language of the project, please feel free to reject the PR. I am also open to any suggestion.

image

Screenshot 2023-03-27 at 20 47 27

Checks

  • In case of new feature, add short overview in docs/<corresponding file>
  • Tested changes

@reactorcoremeltdown reactorcoremeltdown changed the title Moved Edit/Delete/Save buttons to the top toolbar Moved Edit/Delete/Save buttons to the navigation bar Mar 27, 2023
@Linbreux
Copy link
Owner

hi @reactorcoremeltdown, love to hear that. This a handy feature! Would it be possible to move the searchbar back to the left in desktop mode and the icons to the right? Maybe it's also more logical this order: new, edit, del. Having these buttons in the header is indeed way more logical. Thanks for implementing that!

@reactorcoremeltdown
Copy link
Contributor Author

@Linbreux I tried to move things around as you have suggested. Looks better now?
wiki

@Linbreux
Copy link
Owner

I like it! Only the collapsed navbar looks a bit cluttered. I also tried to fix it for the pas 2 hours, bu can't seemed to get it right...

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.

None yet

2 participants