Adds an explanation to the quit page #218

Merged

laurensmartina merged 2 commits from improve-quit-texts into master 2019-04-25 09:31:37 +00:00

Conversation 8 Commits 2 Files changed 3 +9 -6

laurensmartina commented 2019-02-23 15:11:04 +00:00 (Migrated from github.com)

Copy link

At the quit page, it was unclear what the user could expect.
This fixes that problem.

At the quit page, it was unclear what the user could expect. This fixes that problem.

matthijskooijman (Migrated from github.com) reviewed 2019-02-23 15:11:04 +00:00

dianawi (Migrated from github.com) reviewed 2019-02-23 15:11:04 +00:00

giplt (Migrated from github.com) reviewed 2019-02-23 15:11:04 +00:00

tammoterhark (Migrated from github.com) reviewed 2019-02-23 15:11:04 +00:00

matthijskooijman commented 2019-02-23 16:01:30 +00:00 (Migrated from github.com)

Copy link

The commit also changes the button text, @laurensmartina, perhaps you could add that in the commit message?

@all, what do you think of this change/wording? Here's how it looks with this PR:

Personally, I wouldn't mind removing the "Screw you guys..." text, just leave the textbox empty by default?

The commit also changes the button text, @laurensmartina, perhaps you could add that in the commit message? @all, what do you think of this change/wording? Here's how it looks with this PR:     Personally, I wouldn't mind removing the "Screw you guys..." text, just leave the textbox empty by default?

laurensmartina commented 2019-03-30 12:50:02 +00:00 (Migrated from github.com)

Copy link

I think it's fine for now.. It can be further improved if needed.

I think it's fine for now.. It can be further improved if needed.

dianawi commented 2019-04-20 13:03:29 +00:00 (Migrated from github.com)

Copy link

I think it is good, and yes, let's take out the screw you guys text.

I think it is good, and yes, let's take out the screw you guys text.

matthijskooijman commented 2019-04-25 09:31:29 +00:00 (Migrated from github.com)

Copy link

I've removed the "Screw you..." wording, and rebased. Ready to merge now.

I've removed the "Screw you..." wording, and rebased. Ready to merge now.

Sign in to join this conversation.

Reviewers

No reviewers

dianawi

tammoterhark

giplt

matthijskooijman

Labels

Clear labels   

Component: User interface

  

Component: Wymeditor

  

Help wanted

  

Level: Difficult

Structural changes, need discussion

  

Level: Easy

Under 4 hours of work

  

Level: Moderate

Under 16 hours of work

  

Pagetype: Festival

  

Pagetype: Mailinglist

  

Pagetype: Peer reviewed article

  

Pagetype: Text

  

Privacy GDPR AVG

  

status: has conflicts

  

Status: Needs changes

  

Status: Needs discussion

  

Status: Needs review

  

Status: Ready to merge

  

Status: Waiting for response

  

Type: Bug

  

Type: Enhancement

  

Type: Question

  

Usecase: De Stadsbron

  

Usecase: Koppelting

  

Usecase: MeetjeStad

  

Value: Coders

Enhancement of codebase, API, documentation

  

Value: Security

  

Value: Users

Improve user experience, add new features

  

Value: Visitors

Improve navigation, styling

No labels

Component: User interface

Component: Wymeditor

Help wanted

Level: Difficult

Level: Easy

Level: Moderate

Pagetype: Festival

Pagetype: Mailinglist

Pagetype: Peer reviewed article

Pagetype: Text

Privacy GDPR AVG

status: has conflicts

Status: Needs changes

Status: Needs discussion

Status: Needs review

Status: Ready to merge

Status: Waiting for response

Type: Bug

Type: Enhancement

Type: Question

Usecase: De Stadsbron

Usecase: Koppelting

Usecase: MeetjeStad

Value: Coders

Value: Security

Value: Users

Value: Visitors

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

harmen/hypha!218

No description provided.