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

Improve "Delete link" phrasing #87

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

patcon
Copy link

@patcon patcon commented Apr 10, 2015

https://imgur.com/zmUvT8w

My assumption would be that "delete link" hid the link from my screen, not that it removed the pastebin. I was a little confused when I clicked that on my first try, and my new paste was unreachable.

Perhaps "Delete data"?

@Mikaela
Copy link

Mikaela commented Apr 11, 2015

I haven't ever had confusion with what "delete link" does, but maybe it could be more clear.

Perhaps "Delete data"?

How about "Delete paste" or "Remove paste"?

@patcon
Copy link
Author

patcon commented Apr 11, 2015

Sure, I'm easy-going, so whatever you think best

I considered "paste", but thought it might be needlessly confusing for non-native-English speakers (verb + verb)

EDIT: My understanding of UX practices is that "remove" is the word preferred when the underlying data is not affected (ie. cosmetic changes), and "delete" is preferred when it is (ie. permanent changes)

@Mikaela
Copy link

Mikaela commented Apr 11, 2015

I see. It's probably best to just wait for @sebsauvage .

@Hexalyse
Copy link

Hexalyse commented Aug 7, 2015

No commit for 2 years. Better face the facts : this project is dead, sebsauvage won't merge all this pulls requests, ever.

@ulikoehler
Copy link

I'd love to disagree, but unfortunately at least for now this is true...

@equinox
Copy link

equinox commented Aug 7, 2015

Time for a full-on community fork? ;)

@Draky50110
Copy link

@equinox that's a good idea, as done for Shaarli ;)
Other forks have some interesting tweaks.

@elrido
Copy link

elrido commented Aug 16, 2015

Ok, lets do it! I finally updated my fork to be compatible with the 0.19 release of Seb and created some documentation at https://github.com/elrido/ZeroBin/wiki/Development. I'll be taking your issues, ideas and pull requests over in my repo.

@Hexalyse
Copy link

@elrido, it could be a great idea to merge the big PR #63 made by @ulikoehler into your fork, so we can all start contributing on a clean and configurable zerobin. (I'd like to take the PR concerning the password protection of pastes, which I previously used with slight modifitications, and consider merging it into your repo)

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.

7 participants