Automattic / jetpack

Security, performance, marketing, and design tools — Jetpack is made by WordPress experts to make WP sites safer and faster, and help you grow your traffic.
https://jetpack.com/
Other
1.59k stars 799 forks source link

Iteration of Tiled Gallery settings #7576

Open MichaelArestad opened 7 years ago

MichaelArestad commented 7 years ago

Currently Tiled Gallery is connected to Photon in settings. It looks like this: image

After some discussion with @rickybanister, we think it would be a good idea to move it and provide access to a suboption that is missing from Jetpack Settings, yet available in wp-admin.

It would look like this: image

We would also need to remember to update the search keywords and one-way updates for tiled galleries.

We should also mirror this change in Calypso. CC @tyxla @oskosk

rickybanister commented 7 years ago

That's so much more clear.

Also cc @dereksmart

MichaelArestad commented 7 years ago

Benefits:

danjjohnson commented 7 years ago

Maybe this should be a separate issue, but...

image

The second line says

Show photo metadata (Exif) in carousel, when available

But the first line doesn't mention a carousel, so it's kinda confusing.

Should we change the first line to:

Display images and galleries in a gorgeous, full-screen carousel

rickybanister commented 7 years ago

Good catch @danjjohnson. We should also remove the word 'gorgeous'—that editorialization didn't make it into the wordpress.com version of these settings and doesn't add much. It's merely our opinion that it's gorgeous.

stale[bot] commented 6 years ago

This issue has been marked as stale. This happened because:

No further action is needed. But it's worth checking if this ticket has clear reproduction steps and it is still reproducible. Feel free to close this issue if you think it's not valid anymore — if you do, please add a brief explanation.

tyxla commented 6 years ago

This could still be beneficial.

stale[bot] commented 5 years ago

This issue has been marked as stale. This happened because:

No further action is needed. But it's worth checking if this ticket has clear reproduction steps and it is still reproducible. Feel free to close this issue if you think it's not valid anymore — if you do, please add a brief explanation.

jeherve commented 5 years ago

Related: #10753

stale[bot] commented 5 years ago

This issue has been marked as stale. This happened because:

No further action is needed. But it's worth checking if this ticket has clear reproduction steps and it is still reproducible. Feel free to close this issue if you think it's not valid anymore — if you do, please add a brief explanation.