#6479 closed enhancement (fixed)
Encourage people to change default tagline
Reported by: | markjaquith | Owned by: | audrasjb |
---|---|---|---|
Milestone: | 6.1 | Priority: | normal |
Severity: | normal | Version: | 2.5 |
Component: | Administration | Keywords: | has-patch needs-user-docs has-unit-tests |
Focuses: | Cc: |
Description
A lot of people don't change the default tagline, especially if using a theme that doesn't display it. But it still displays in their feed. We should check to see if they still have the default tagline and if so, put a little contextual help in Settings - General that (nicely) encourages them to change it.
Props to Chris Silver Smith for bringing this issue up at WordCamp Dallas.
Attachments (4)
Change History (68)
#4
@
15 years ago
- Keywords close removed
- Owner set to markjaquith
- Status changed from new to assigned
This sounds like a good idea.
Maybe it should be promoted to an Install time configurable?
#6
@
13 years ago
- Keywords has-patch added; needs-patch removed
Attached patch adds the tagline as an install time option. Patch also sets the default tagline to (i.e. none) (#18552).
#9
@
12 years ago
Two cents: In 6479.patch the new argument to wp_install
should be the last in the list, otherwise you might break existing calls to wp_install
. It should also be an optional argument. In schema.php
the empty string should not be passed through the translation functions, in fact I don't think it should be an empty string at all. The default value for the tagline should remain as is -- the ticket says "encourage" people to change the default tagline, not "force" :)
#11
@
12 years ago
The empty string came out of #18552. A more recent suggestion: ticket:18552:10.
#13
@
9 years ago
- Keywords ux-feedback added
I don't think this is just limited to the tagline. Many users never edit the time settings to their correct time. Maybe the UX team has an idea about how to improve the settings so users are encouraged to go through all settings
#14
@
9 years ago
- Keywords has-patch added; needs-patch removed
The recent patch above follows the notes from @kovshenin.
#15
@
8 years ago
Why don't we direct users to the settings page after the installation process? Like other websites do after registering. Why should the dashboard be the starting point and not the settings page first?
#16
@
8 years ago
To me, the dashboard makes sense to be the place if the information on it leads people to discovery as part of the NUX. This is actually something I think could be improved, that's outside the scope of this ticket though I would suggest. I think having encouragements like this make sense, even if we have a NUX flow that includes a prompt, people skip flows.
#17
@
8 years ago
@karmatosed
So it might more sense for having reminders on the dashboard, if a user has not updated the site tagline from default or add a new page or post from default content.
#18
@
8 years ago
@lukecavanagh if that's the end of the NUX flow then yes. I do note that whilst I don't know if planned I think the next release was focusing on NUX a little. This could be something to bring to that.
This ticket was mentioned in Slack in #design by karmatosed. View the logs.
7 years ago
#22
@
7 years ago
Taglines feel pretty anachronistic at this point, especially as WordPress is used for many different kinds of sites. I'd be a +1 for removing the default tagline, as suggested in #38316.
This ticket was mentioned in Slack in #design by boemedia. View the logs.
6 years ago
#24
@
6 years ago
We discussed this ticket in today's design triage. Summarizing, we can conclude 3 things:
- Not all themes display the tagline
- Not everyone changes the default tagline
- When people don't see the tagline in their theme, they don't realise it's still visible in feeds (and search results as well?)
I'm not even sure if a lot of website owners want a tagline in the first place, or have one at all. This could typically be a maybe later and then be forgotten.
I'd suggest making this a 2-step process while installing WP, but check if people see these fields while they use hosting services to install WP:
- ask if people want a tagline on their website (+ info: what is a tagline, where is it displayed, examples, why have one -> maybe link to some blog post on .org?). If checked yes, then:
- open conditional field where people can fill out the tagline.
This ticket was mentioned in Slack in #design by karmatosed. View the logs.
5 years ago
#27
@
4 years ago
- Severity changed from normal to trivial
Any thoughts on changing the default tagline to an empty string?
#28
@
3 years ago
- Milestone changed from Future Release to 6.0
- Severity changed from trivial to normal
I think now is a good time to revisit this. Removing the Customizer from block themes likely will increase the number of people who have the default tagline on their site without even knowing about it.
The current text can remain in Core, but it might work better as a default field value only within the Customizer (if possible) for people still using that. Or it could be an example below the field in the Customizer and/or Settings.
If a (visible) Tagline field becomes part the installation process, that would involve editing two network files (site-new.php and wp-signup.php) besides install.php.
If we also want to encourage changing the tagline for existing sites, we could reopen #38316 to set the option to an empty string for new installations, and then address existing installations on this ticket separately. (That ticket is very closely related to this one, but it's not an exact duplicate.)
#29
@
3 years ago
- Keywords needs-refresh 2nd-opinion added
I do think the easier solution is to replace the "Just another WordPress site" value with a placeholder. By doing this, we keep this sentence which is in my opinion part of the WordPress history —by doing so, it would at least appear on the Settings screen, so we keep this signature sentence somewhere on the admin— but the value is empty by default for new installs.
Thoughts?
@
3 years ago
tagline placeholder in General Settings, empty string for standard blogdescription
option
#30
@
3 years ago
Yes, adding a placeholder in General Settings is easier than editing the Customizer or installation forms. If that is a good approach, the patch still will need updating for unit tests (including at least wpGetDocumentTitle.php).
#31
@
3 years ago
In my opinion, 6479.3.patch
is the best approach 👍
By the way, I think it would be nice to get a second opinion from historical committers before shipping it :)
#32
@
3 years ago
- Keywords needs-testing added; needs-refresh removed
Thanks for the patch @sabernhardt. Pushing it as a pull request now, to make sure it passes Unit tests (I feel a bit hesitant about some export tests).
This ticket was mentioned in PR #2230 on WordPress/wordpress-develop by audrasjb.
3 years ago
#33
Trac ticket: https://core.trac.wordpress.org/ticket/6479
#34
@
3 years ago
- Keywords needs-unit-tests added
That's what I thought: tests are failing. Updating your patch accordingly :)
This ticket was mentioned in Slack in #core by chaion07. View the logs.
3 years ago
This ticket was mentioned in Slack in #core by chaion07. View the logs.
3 years ago
#37
follow-up:
↓ 38
@
3 years ago
- Keywords needs-refresh added; 2nd-opinion removed
- Owner markjaquith deleted
This was discussed today in a triage session.
There's still a little to do prior to release but we can keep it on the milestone for now.
- There are a few unit tests failing relating to
the_title
- If not already, adding the site tag to the installation screens would be good too
- Ensure the tagline is optional
props: @hilayt24, @chaion07, @mehedi890, @markparnell for contributing to discussion
#38
in reply to:
↑ 37
@
3 years ago
Replying to peterwilsoncc:
- If not already, adding the site tag to the installation screens would be good too
Are you sure? Adding items to the installation screen is a sensible change since each addition to this screen could be considered as a threat for the "5 mins" installation process. I think this specific proposal needs to be discussed a lot more before implementing it :)
#39
@
3 years ago
@audrasjb Fair, leave that for now and it can be discussed to consider further implications.
This ticket was mentioned in Slack in #core by chaion07. View the logs.
3 years ago
#42
@
3 years ago
Per the discussion in the bug scrub, I'm going to work on the refresh of the PR to target these outstanding issues:
- There are a few unit tests failing relating to
the_title
. - Ensure the tagline is optional.
- This comment on the PR regarding the unit tests.
I'll post a comment when the PR is ready for review.
#43
@
3 years ago
- Keywords needs-testing removed
- Owner set to audrasjb
- Status changed from assigned to reviewing
Thanks @costdev for taking care of this! Self-assigning to make sure it's handled in time for beta 1. A new PR would be very appreciated, but if you don't have the time to work on it, let me know :)
This ticket was mentioned in Slack in #core by chaion07. View the logs.
3 years ago
#47
@
2 years ago
@costdev , have you done any more work on this recently? I am happy to help with the unit tests if still needed.
#48
@
2 years ago
Hi @martinkrcho! I haven't done work on this recently, so feel free to jump in. If that changes, let me know and I'll get back onto this one.
#49
@
2 years ago
I think we can add an admin notice if someone doesn't add a tagline. I will submit a patch for this.
This ticket was mentioned in PR #3050 on WordPress/wordpress-develop by costdev.
2 years ago
#50
- Keywords has-unit-tests added; needs-unit-tests needs-refresh removed
This PR refreshes PR 2230 to fix failing tests and add another testcase as requested by @peterwilsoncc in this comment.
Trac ticket:
#51
@
2 years ago
@audrasjb @peterwilsoncc PR 3050 is ready for review.
Note: Separate from the PR, 6479.4.patch has been added by @rafiahmedd and needs discussion.
#52
@
2 years ago
Thanks for updating the tests!
About 6479.4.patch:
Having an empty tagline is not an error.
If we want an admin notice to recommend setting a tagline, it:
- Must verify that the user has the
manage_options
capability to change site settings. - Needs to be permanently dismissible.
- Could have the
notice-info
(blue) styling. - Needs meaningful text in the link (something like "create a site tagline" instead of "click here").
sprintf( __( 'Your site does not have a tagline. If you would like, you could <a href="%s">create a site tagline</a>.' ), admin_url( 'options-general.php' ) )
#53
@
2 years ago
Thanks for the feedback @sabernhardt!
On the admin notice, is there support for @rafiahmedd's proposal to introduce one?
If so:
- The notice would appear on every admin page except
options-general.php
, correct?
- The initial user with the
manage_option
capability would make the decision for the whole site, correct?
This touches on the earlier discussion about the install screen. If we don't want to add an additional option to the install screen, then we need to consider how the notice is permanently dismissible.
Two options are:
- Similar to the
default_password_nag
, asite_tagline_nag
option could be set totrue
inwp_install()
. This would be set tofalse
upon clicking a "No thanks, don't ask me again" link in the notice. - The
disable_site_tagline_nag
option is only created when the user clicks "No thanks, don't ask me again", and is set totrue
.
Thoughts?
#54
@
2 years ago
I'm not sure an admin notice is really needed here as it's up to people to set up a tagline or not. Not having a tagline is not a mistake in itself.
Indeed, it is not mandatory to use it by default: in bundled themes, it is appended to the <title>
tag and used in the header of the website, but that's optional.
#55
@
2 years ago
Thanks @audrasjb! I think that just leaves PR 3050 for this ticket. Do you have bandwidth for this one in your review list?
2 years ago
#57
Closed in favor of https://github.com/WordPress/wordpress-develop/pull/3050
#58
@
2 years ago
- Keywords commit added
Looks good to me. Marking for commit
… and committing right now.
2 years ago
#61
loool wrong copy paste 😂 😂 😂
Committed in https://core.trac.wordpress.org/changeset/53815
#62
@
2 years ago
- Keywords commit removed
I suggested adding a paragraph about the site tagline on the About page (ticket:56357#comment:19).
@webcommsat asked about other documentation:
Is there a dev note that will cover Ticket #6479? The change is quite significant and it would be good for it to be highlighted in other places too including docs and support. I will highlight it to colleagues I'm working alongside in the release docs team. I will raise it in Training too as from memory some items on Learn WordPress.org relate to the site tagline. Thanks.
I had not planned to write a special dev note about this change, though mentioning it in the Field Guide may help.
This ticket was mentioned in Slack in #core by costdev. View the logs.
21 months ago
#64
@
14 months ago
HelpHub was updated to note the new empty-string default: https://wordpress.org/documentation/article/settings-general-screen/
suggesting wontfix