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

Use file name instead of first line as title [$20] #190

Closed
godfuture opened this issue Apr 17, 2018 · 18 comments · Fixed by #474
Closed

Use file name instead of first line as title [$20] #190

godfuture opened this issue Apr 17, 2018 · 18 comments · Fixed by #474
Labels
bounty There is a bounty for this issue enhancement New feature or request needs discussion Need to clarify if and how we should implement this
Milestone

Comments

@godfuture
Copy link

godfuture commented Apr 17, 2018

Using the first line as title has the drawback that information is displayed redundant. As space on screen is valuable, on mobile devices even more, it makes sense to expose the title separately from the body and persist it as filename. This way there is no longer redundancy. This still works well with basic text editor and files app syncing to desktop.


Did you fix this issue? Go claim the $20 bounty on Bountysource.

@stefan-niedermann
Copy link
Member

#116
owncloud/notes#198

@godfuture
Copy link
Author

Understood. But I would want to constructively disagree:

  • Microsoft and others still know a "title" for their notes. The title is on the very top and underlined.
  • Files have names and this app is file based. Why not letting the title to be the file name. In most cases the title is a good summary.
  • File name and first line showing totally the same content is also not very reasonable.

To be honest, I guess there is more PRO than CON for it...

@camilasan camilasan added needs discussion Need to clarify if and how we should implement this feature request labels May 4, 2018
@natrius
Copy link

natrius commented May 27, 2018

I do also disagree. A separate field for a Title would be nice, it's always bugging me. Using the title as filename is perfectly fine. :)

@jancborchardt
Copy link
Member

Especially with the mobile app around and the issues with it:

  • Single note widget shows redundant first line
  • Changing first line can cause sync issues
  • Viewing a note shows the filename already in the header

… I’m thinking again about if it might be a good idea to split first line and file name.


We could still handle it exactly like now on note creation: By default the first line you type is used as filename.

But once created, when you change the first line, the note title is not changed. For that you have to use the 3-dot menu in the note list to »Change filename«.

What do you think @nextcloud/notes @Henni @camilasan @stefan-niedermann?

@jancborchardt jancborchardt added enhancement New feature or request and removed enhancement New feature or request feature request labels Jun 5, 2018
@korelstar
Copy link
Member

I'm fine with that, although I see other things that would be more important. But if someone implements this for the frontend, I'm willing to change the 3rd-party (notesapicontroller) in order to be compatible to both type of apps: using the old style (no separate title) and the new style (separate title).

@frederiiiic
Copy link

I'm sometimes editing my *.txt files on my computer (NotePad++), sometimes with the web app, and sometimes with the nextcloud-notes app. And i'm frequently expériencing some troubles with file names.

@jospoortvliet jospoortvliet changed the title Use file name instead of first line as title Use file name instead of first line as title [$10] Aug 25, 2018
@jospoortvliet jospoortvliet added the bounty There is a bounty for this issue label Aug 25, 2018
@korelstar
Copy link
Member

@frederiiiic Which kind of troubles do you are talking about?

@frederiiiic
Copy link

For exemple when i'm changing the first line, the nextcloud app change the name of the file. And if i've changed it on the txt file of my computer, this will be deleted.
I just have to known that but i find this logic weard. Using file name, look more familiar, natural and simple to me, but maybe with differents using it's not...

@godfuture
Copy link
Author

Maybe we might introduce a slider to decide whether the first line to be interpreted as filename or not?!

@stefan-niedermann
Copy link
Member

IMHO: An option would require to maintain two completely different views. We better should decide for one default view and only provide this.

@godfuture
Copy link
Author

godfuture commented Feb 15, 2019

As this project seems dead (no commits for quite a long time), I want to let you know how I solved it. With two apps, you get close to the Nextcloud nodes app. Install Denkzettel and Folder Sync. You have to add your Nextcloud as webdav server and add a sync folder.

With the correct options set, your experience comes close to the Nextcloud solution. No nodes app on server needed. Good luck!

@korelstar
Copy link
Member

Thanks for sharing your experiences :-)

However, this project is far away from being dead. Instead, it undergoes a big modernization, see #290. After that, this issue is one of the issues with higher priority. You can speed this up by increasing the bounty on this, bye the way.

@godfuture
Copy link
Author

What is the state? Is this still considered for any of the milestones?

@korelstar
Copy link
Member

You're right, I'm adding this to the next milestone. For the GUI, this will depend on #373 (and therefore on the next release of nextcloud-vue); for the third-party API this depends on #311.

@korelstar korelstar added this to the 3.1.0 milestone Sep 19, 2019
@trevorparsons
Copy link

The main issue for me with the first line of the note being used for the filename is that this very often results in a filename which contains spaces. This is obviously not optimum for when one wants to operate on the file in the command-line interface.

@RahulxPaul

This comment has been minimized.

@michaelnew
Copy link

I know it's not much, but I threw another $10 at this. Maybe @godfuture could update the title?

@stefan-niedermann stefan-niedermann changed the title Use file name instead of first line as title [$10] Use file name instead of first line as title [$20] Nov 4, 2019
@korelstar korelstar modified the milestones: 3.1.0, 3.2.0 Dec 14, 2019
@korelstar korelstar modified the milestones: 3.2.0, 3.3.0 Mar 13, 2020
@korelstar
Copy link
Member

Okay, people. The implementation is finished (see #474), so please test if it is like you expected (checkout the branch edit-title and run make to compile JavaScript). I will write the documentation for the new API version in the meanwhile.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bounty There is a bounty for this issue enhancement New feature or request needs discussion Need to clarify if and how we should implement this
Projects
None yet
Development

Successfully merging a pull request may close this issue.