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

Change design of compose form in web UI #28119

Merged
merged 6 commits into from Jan 25, 2024
Merged

Conversation

Gargron
Copy link
Member

@Gargron Gargron commented Nov 28, 2023

One of our testers said that the compose form had an outdated "Windows 7" look about it, so I asked our designer what we could do about it. Also, now it's dark in dark mode, and I've renamed the confusing "Unlisted" into a slightly less misleading "Quiet public".

Editing a post:

Screenshot 2024-01-23 at 08-35-45 Localhost

Replying to a post:

Screenshot 2024-01-23 at 08-35-29 Localhost

Adding a content warning:

Screenshot 2024-01-23 at 08-35-10 Localhost

Adding media:

Screenshot 2024-01-23 at 08-34-56 Localhost

Adding a poll:

Screenshot 2024-01-23 at 08-34-29 Localhost

Empty:

Screenshot 2024-01-23 at 08-34-05 Localhost

Selecting post visibility:

grafik


Fixes MAS-206

Additionally:

@Gargron Gargron added the ui Front-end, design label Nov 28, 2023
@connie-e
Copy link

The composer looks cramped with the post button being with the icons.

@Gargron Gargron force-pushed the feature-redesign-compose branch 2 times, most recently from e63a862 to 07cfc2f Compare December 1, 2023 03:24
@Andre601
Copy link

Andre601 commented Dec 1, 2023

I personally feel like the compose button looks horrible on the side, no matter how much you clean it up and improve it.

It always feels squeezed in and as if its potential isn't used here.

Why not have a box at the top of the timeline where clicking it expands it, to then write in and post?

Having to the side feels outdated and like an after thought rather than an actual feature...

Like put a new user in front of a personal timeline and ask them what that thing on the left side does. Chances are they either don't know or guess an incorrect behaviour.
With the compose box on top of the timeline a user would fairly quickly piece together that it is connected to said timeline.

And with the left sidebar freed up a bit could extra stuff such as instance info (i.e. their description) be provided.

Either way, this design is just awful and the positioning makes no sense. And that comes from a regular Mastodon User.

@renchap
Copy link
Sponsor Member

renchap commented Dec 1, 2023

This is a work in progress and we are working on improving this with our designer.

We do not plan large changes to the compose box at the moment (because they would require a large redesign), like moving it elsewhere on the screen, but the final design will be less cramped.

@joenepraat
Copy link
Contributor

Looks good. But what I really like to see is to sync it with the rest of the dark theme. Even the text field can be dark.

I also like to have an option to pop-out the compose UI as a large modal overlaying the rest of the interface and/or to open it in a new tab.

@Gargron Gargron force-pushed the feature-redesign-compose branch 5 times, most recently from b847f26 to 58ebb1c Compare December 8, 2023 05:08
@Gargron Gargron force-pushed the feature-redesign-compose branch 3 times, most recently from 85105ab to d9c42e4 Compare December 15, 2023 07:18
@Gargron Gargron marked this pull request as ready for review December 15, 2023 07:24
@jmking-iftas
Copy link

jmking-iftas commented Dec 15, 2023

I feel like the privacy and language "public" and "english" should be inverted to white on blue along the same lines as a selected post type. In these examples they look more like content tags. If they were inverted so they are "selected" (and assuming when i tap ore click I get the drop down options which would not be selected/blue on white) it might help them.

I like graying out (and disabling?) the eg media attachment when poll is selected or vice versa, nice touch, better than hiding it IMO. Will the grayed out get a desc of "a poll cannot be added to posts with uploaded media" type language? Should compose-form__upload-button be white on blue if at least one item has been uploaded?

Moving privacy to include the word (public/followers-only etc) is an upgrade (maybe lose the icons altogether?) and having language next to it also a plus. I see some logic as to what languages show up there, but in general I'd want to be sure the languages I use mostly appear at the top of that list, or restrict it if I've set language filters to only show me eg ten languages, those are the ten that should be in this list.

Should these two be separated from the compose area? When I add a poll I get a HR feature in compose-form__uploads-wrapper, maybe something similar to tease these two post configurations apart from the post content itself? ( I note the hr has been removed in these mockups, I think it should be reviewed for helpfulness understanding the free text compose area vs fixed items)

Changing the button text to reply or update when relevant is a great idea, and will help folks who think they're replying to a toot realise it's unattached. There will be some holdouts (probably me included) who still want the "post" to read "toot" instead, which may impact the update and reply option due to hacky CSS workarounds, but in general having context-sensitive button text is an improvement.

The emoji icon was likely moved for good reasons, but I'm curious why it's no longer up in the content area - that's where emojis go. Having this icon down with the "post configuration stuff" makes it seem like I need to set this post's mood or something, not insert emoji.

Edit, forgot to mention, very much like that CW is now joined to the compose box and not separated, not sure about the caution decoration, not all CW usage is actually a warning, sometimes it's a set up for a joke or a topic or subject heading, but that's probably a minor concern

@Gargron Gargron added the build-image Build a container image for this PR label Dec 18, 2023
@vmstan
Copy link
Contributor

vmstan commented Dec 18, 2023

Edit button coloring is inverted on High Contrast theme, icon coloring is also inconsistent.

Screenshot 2023-12-18 at 9 32 22 AM Screenshot 2023-12-18 at 9 31 49 AM

@ClearlyClaire
Copy link
Contributor

This looks more modern overall and I'm happy language and visibility options are visible and spelled out, and I'm happy about the single/multiple poll options being explicit now!

This feels a bit cramped overall, but the two-line fallback looks very good!
image

The new reply indicator has potential, but I feel the inconsistent layout confusing (I think the display name and handle should be on two lines):
image

The contrast overall seems off, especially in the CW field:
image

The lack of border or contrast between the dropdowns and the composer feels confusing as well:
image

Margins are missing from the mobile UI:
image

… as well as the advanced interface:
image

@vmstan
Copy link
Contributor

vmstan commented Dec 18, 2023

Yeah can confirm mobile margins are missing.

@Andre601
Copy link

I think the CW needs a bit more work. This yellow-black stripe patterns doesn't fit the overall style from what I can tell and a simple yellow border would do the trick too.

Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@nileane
Copy link

nileane commented Jan 25, 2024

Something I just noticed while working on adapting my custom theme to the new compose form, the selected visibility icon has an "icon-undefined" class regardless of what visibility is selected (which is problematic in my case for assigning a custom icon to each visibility setting).

Safari - 2024-01-25 à 16 24 2@2x

For instance, the lock icon for the "Followers" visibility should have an "icon-lock" class, just like it does in the dropdown:

Safari - 2024-01-25 à 16 28@2x

Gargron and others added 2 commits January 25, 2024 16:34
Co-authored-by: Claire <claire.github-309c@sitedethib.com>
@Gargron Gargron added this pull request to the merge queue Jan 25, 2024
Merged via the queue into main with commit 6936e5a Jan 25, 2024
45 checks passed
@Gargron Gargron deleted the feature-redesign-compose branch January 25, 2024 15:47
@renchap
Copy link
Sponsor Member

renchap commented Jan 25, 2024

This has been merged with an updated label for the previous "Unlisted" option based on the various feedback we received.

This may be changed again after gathering more data and ideas, but we wanted to have the composer update merged so we can have wider testing on it, and any other change will be done in a further PR.

If you want to participate to this discussion about the "Unlisted" visibility, please use the ad-hoc discussion: #28887

@nclm
Copy link
Contributor

nclm commented Jan 26, 2024

Tiny bug report: clicking on the emoji icon when the emoji picker is opened should close the picker (in the same way clicking again the poll or cw icon closes the poll or cw fields). Right now, clicking the emoji icon again seems to reopen the emoji picker which is already opened.

@Ember-ruby
Copy link

i don't get the change of moving the publish button into the compose box, it just seems to needlessly cramp both it and the compose box

@omentic
Copy link

omentic commented Apr 14, 2024

Hello, with the addition of scary orange-and-black tape to the content warning field, I have anecdotally noticed a significant reduction in their use. I happen to think that content warnings are extremely useful, as content warnings and also as titles. Having DANGER ALERT style CSS when composing them discourages new users from using them, I think, and I think that's not something to be discouraged.

I would appreciate it if this style change was reverted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment