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 icons in web UI #27385

Merged
merged 17 commits into from
Oct 24, 2023
Merged

Change icons in web UI #27385

merged 17 commits into from
Oct 24, 2023

Conversation

Gargron
Copy link
Member

@Gargron Gargron commented Oct 12, 2023

We're finally replacing Font Awesome with Material icons, consistent with our Android app. For now this is a pretty straightforward mapping of old icons to new icons, but Material gives us more possibilities (e.g. filled in for active, outline for inactive) that I imagine we'll explore gradually afterwards.

image


Fixes MAS-166, fixes #26617, fixes #1469

@Gargron Gargron added the ui Front-end, design label Oct 12, 2023
@Gargron Gargron marked this pull request as draft October 12, 2023 20:22
@Gargron Gargron force-pushed the feature-icons branch 3 times, most recently from 06fd4b9 to 4ef1b30 Compare October 12, 2023 22:32
@Gargron Gargron marked this pull request as ready for review October 17, 2023 20:01
ykzts
ykzts previously approved these changes Oct 18, 2023
@vmstan
Copy link
Sponsor Contributor

vmstan commented Oct 18, 2023

Gave this PR a test run, and the icon alignment in post views is way off in Safari.

CleanShot 2023-10-18 at 16 27 39@2x
CleanShot 2023-10-18 at 16 26 02@2x
CleanShot 2023-10-18 at 16 25 22@2x
CleanShot 2023-10-18 at 16 32 03@2x

It appears fine in Chrome and Firefox

@vmstan
Copy link
Sponsor Contributor

vmstan commented Oct 18, 2023

image

More problematic on mobile.

@github-actions
Copy link
Contributor

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

@Gargron Gargron force-pushed the feature-icons branch 4 times, most recently from 8641338 to ffc96ec Compare October 20, 2023 00:02
@github-actions
Copy link
Contributor

This pull request has resolved merge conflicts and is ready for review.

@vmstan
Copy link
Sponsor Contributor

vmstan commented Oct 20, 2023

Safari issue is fixed.

@vmstan
Copy link
Sponsor Contributor

vmstan commented Oct 20, 2023

image

Reply content with client info appears misaligned for longer client names.

@vmstan
Copy link
Sponsor Contributor

vmstan commented Oct 20, 2023

Client information appears fixed now.

@ClearlyClaire
Copy link
Contributor

The dropdown menu icon is definitely misaligned. Also, all icons on posts seem slightly too small. Finally, animations (fav icon and boost icon) are missing. The outline VS filled distinction is welcome, but it's not there for boosts, which has also lost its variants and animations.

I'm also running into an infinite loop in the advanced interface.

@vmstan
Copy link
Sponsor Contributor

vmstan commented Oct 20, 2023

I'm also running into an infinite loop in the advanced interface.

Was just coming here to say this.

@Gargron
Copy link
Member Author

Gargron commented Oct 20, 2023

Also, all icons on posts seem slightly too small.

Material Symbols include padding in the SVGs which means the actual shapes are smaller than the equivalent Font Awesome symbols at the same container size. However, I have adjusted the sizes to be closer to what they used to be. I believe any remaining impression of smallness is just from them being a bit thinner design-wise.

Finally, animations (fav icon and boost icon) are missing

I won't re-add them. Favourite icon is about to become a heart, so spinning will make no sense anymore. There's a ticket for Sam to create interaction animations for the iOS/Android apps, once that exists I imagine we'll be able to port it back to web.

@Gargron Gargron force-pushed the feature-icons branch 3 times, most recently from 922982a to c3f6976 Compare October 21, 2023 02:39
@Gargron Gargron added this pull request to the merge queue Oct 24, 2023
Merged via the queue into main with commit 134de73 Oct 24, 2023
64 checks passed
@Gargron Gargron deleted the feature-icons branch October 24, 2023 17:45
renchap added a commit to renchap/mastodon that referenced this pull request Oct 24, 2023
@renchap renchap mentioned this pull request Oct 24, 2023
zunda pushed a commit to zunda/mastodon that referenced this pull request Oct 24, 2023
Co-authored-by: Renaud Chaput <renchap@gmail.com>
zunda added a commit to zunda/mastodon that referenced this pull request Oct 24, 2023
@colinstu12
Copy link

colinstu12 commented Oct 29, 2023

I won't re-add them. Favourite icon is about to become a heart, so spinning will make no sense anymore. There's a ticket for Sam to create interaction animations for the iOS/Android apps, once that exists I imagine we'll be able to port it back to web.

@Gargron
Where was it discussed or decided that the star should become a heart? The star is a lot more generic of an interaction than a heart is (that indicates liking/loving something). Not exactly the nicest thing to use with posts that contain bad or sad news/topics, etc.
Twitter made this same move too long ago and didn’t seem appropriate either.

@edoswald
Copy link

I understand we're changing the star to a heart, but in a thread on this on Mastodon came up with some interesting real-world data on the "star." It seems that people are using it for a variety of uses, some duplicating pre-existing features, like bookmarks. Changing it to a heart will also likely change its usage since the heart is used for status reactions on most other networks, so newbies will likely do it here.

Maybe before changing the star to a heart, maybe taking a look at the functionality the star provides, and rethinking it? I know this is a simple change, but it may result in some unintended consequences from an interaction standpoint, some of which may not be what the devs intended.

I think the base problem is nobody knows exactly what the star is for, or does. I'd be strongly in favor of considering a change to how the star works before such a switch.

@edoswald
Copy link

I won't re-add them. Favourite icon is about to become a heart, so spinning will make no sense anymore. There's a ticket for Sam to create interaction animations for the iOS/Android apps, once that exists I imagine we'll be able to port it back to web.

@Gargron Where was it discussed or decided that the star should become a heart? The star is a lot more generic of an interaction than a heart is (that indicates liking/loving something). Not exactly the nicest thing to use with posts that contain bad or sad news/topics, etc. Twitter made this same move too long ago and didn’t seem appropriate either.

I think it's fine. Meta decided to overcomplicate it with four other options. I also think the average consumer is mature enough to understand that the like is for the content, not necessarily the topic.

@brendanjones
Copy link

Please don’t change the star to a heart. Every time I go to favourite a post about a negative topic I’ll feel resistance. It won’t be the end of the world, but I won’t like it.

@ronilaukkarinen
Copy link
Contributor

I don't know if this is the place to chime in about the stars vs. hearts, but I think it's fine. I really use the the star like a heart already (1. Acknowledged, I have read your post, 2. Sending love/understanding/agreement).

@edoswald
Copy link

edoswald commented Oct 29, 2023 via email

@Squaredude
Copy link

Adding my two cents on this matter.

My biggest concern with the usage of a "heart" icon is that it predisposes a certain mentality/emotion. As others have said, one can feel awkward "liking" / "hearting" ("loving"?) a post with tragic content.

But, on the abstract side of things, it seems manipulative, like something a for-profit would do.

One thing I've been liking about Mastodon is that the interface and the environment of the Fediverse is not algorithmic, not commercial, and therefore there is no need to encourage the users into having emotional interactions. "Bookmarks", "boosts", "stars"/"favorites", "posts"... these are neutral terms, they don't invoke emotions or predispose users to interact.

I hope the measure can be reconsidered. It's frankly disappointing that this was decided.

@Gargron
Copy link
Member Author

Gargron commented Oct 31, 2023

I think you mean to comment on #27582

mgmn pushed a commit to mgmn/mastodon-kumarinco that referenced this pull request Nov 4, 2023
Co-authored-by: Renaud Chaput <renchap@gmail.com>
audiodude pushed a commit to audiodude/mastodon that referenced this pull request Nov 16, 2023
Co-authored-by: Renaud Chaput <renchap@gmail.com>
lunaisnotaboy added a commit to michiru-me/akkodon that referenced this pull request Nov 21, 2023
commit 59893a4
Merge: 18eacc7 36c8570
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Fri Oct 27 17:45:19 2023 +0200

    Merge pull request glitch-soc#2449 from ClearlyClaire/glitch-soc/merge-upstream

    Merge upstream changes up to 9a3d047

commit 36c8570
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Wed Oct 25 20:01:51 2023 +0200

    Fix HAML linting issue

commit 6262095
Merge: 85d4846 9a3d047
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Wed Oct 25 19:59:17 2023 +0200

    Merge commit '9a3d047f3e604e581e18346424569e28fc9c5b96' into glitch-soc/merge-upstream

commit 9a3d047
Author: Matt Jankowski <matt@jankowski.online>
Date:   Wed Oct 25 09:56:09 2023 -0400

    Run `bin/rails app:update` with Rails 7.1 (mastodon#27522)

commit 235a4cf
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Wed Oct 25 15:55:57 2023 +0200

    Fix batch attachment deletion when using OpenStack Swift (mastodon#27554)

commit a4e6fe3
Author: Renaud Chaput <renchap@gmail.com>
Date:   Wed Oct 25 14:47:37 2023 +0200

    Add missing ban icons (mastodon#27555)

commit 70dbf84
Author: Matt Jankowski <matt@jankowski.online>
Date:   Wed Oct 25 08:38:01 2023 -0400

    Solve simplest haml-lint `Rubocop` lints (mastodon#27529)

commit fa65c82
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Wed Oct 25 14:01:21 2023 +0200

    Fix cache store pool config deprecation warnings (mastodon#27551)

commit c926f5f
Author: Matt Jankowski <matt@jankowski.online>
Date:   Wed Oct 25 08:01:00 2023 -0400

    Fix haml-lint `UnnecessaryStringOutput` for views (mastodon#27531)

commit 85d4846
Author: Renaud Chaput <renchap@gmail.com>
Date:   Tue Oct 24 22:43:55 2023 +0200

    [Glitch] Fix `<ColumnBackButtonSlim>`

    Port cb92cdf to glitch-soc

    Signed-off-by: Claire <claire.github-309c@sitedethib.com>

commit 6e9f89e
Merge: ba527c0 c85e0a6
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Wed Oct 25 12:18:45 2023 +0200

    Merge commit 'c85e0a6b047f1a72ad970bc04c4e1be3f028b3d1' into glitch-soc/merge-upstream

    Conflicts:
    - `app/javascript/packs/admin.jsx`:
      Upstream fixed `@rails/ujs` imports.
      Already taken care of in 8a131fb
    - `app/javascript/packs/public.jsx`:
      Upstream fixed `@rails/ujs` imports.
      Already taken care of in 8a131fb

commit ba527c0
Merge: b19d4eb 134de73
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Wed Oct 25 12:14:24 2023 +0200

    Merge commit '134de736dcbc6aa613fd5aec21d983d92d8b0be8' into glitch-soc/merge-upstream

    Conflicts:
    - `app/javascript/mastodon/features/compose/components/poll_form.jsx`:
      Upstream changed how icons are handled, including on a line modified by
      glitch-soc to bump the number of poll options.
      Applied upstream's change, while keeping the increased number of poll
      options.

commit c85e0a6
Author: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Date:   Wed Oct 25 09:36:07 2023 +0200

    New Crowdin Translations (automated) (mastodon#27544)

    Co-authored-by: GitHub Actions <noreply@github.com>

commit f261b64
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Wed Oct 25 09:16:19 2023 +0200

    Update dependency node to 20.9 (mastodon#27534)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit cec7f0e
Author: Matt Jankowski <matt@jankowski.online>
Date:   Wed Oct 25 03:12:32 2023 -0400

    Remove unused `#error_explanation` id from error partials (mastodon#27536)

commit cb92cdf
Author: Renaud Chaput <renchap@gmail.com>
Date:   Tue Oct 24 22:43:55 2023 +0200

    Fix `<ColumnBackButtonSlim>` (mastodon#27540)

commit f08ca3f
Author: Renaud Chaput <renchap@gmail.com>
Date:   Tue Oct 24 22:43:06 2023 +0200

    Fix missing icon props (mastodon#27539)

commit bc3afb6
Author: Matt Jankowski <matt@jankowski.online>
Date:   Tue Oct 24 16:38:19 2023 -0400

    Update `rails/ujs` delegate method usage (mastodon#27538)

commit b19d4eb
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Tue Oct 24 20:53:58 2023 +0200

    Fix HAML linting issues

commit 8123e86
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Tue Oct 3 13:12:39 2023 +0200

    [Glitch] Fix double scroll bars in some columns in advanced interface

    Port 37bbd3c to glitch-soc

    Signed-off-by: Claire <claire.github-309c@sitedethib.com>

commit b73e4b5
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Tue Oct 24 15:24:57 2023 +0200

    [Glitch] Fix history handling not properly handling states after update to react-router v5

    Port 15182d1 to glitch-soc

    Signed-off-by: Claire <claire.github-309c@sitedethib.com>

commit c6f9909
Author: たいち ひ <taichi221228@icloud.com>
Date:   Tue Oct 24 17:06:14 2023 +0900

    [Glitch] Rewrite `emoji_unicode_mapping_light` to TS

    Port 9482810 to glitch-soc

    Co-authored-by: taichi.fukuda ひ <taichi.fukuda@systemi.co.jp>
    Signed-off-by: Claire <claire.github-309c@sitedethib.com>

commit 4ef66d6
Merge: 3c9c029 15182d1
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Tue Oct 24 20:31:20 2023 +0200

    Merge commit '15182d1e5e595b21c47b48c5d258f30a0251e753' into glitch-soc/merge-upstream

    Conflicts:
    - `.rubocop_todo.yml`:
      glitch-soc had extra ignores.
      Removed them.

commit 3c9c029
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Tue Oct 24 20:23:46 2023 +0200

    Fix ActiveRecord::Migrator monkey-patch

commit 8a131fb
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Tue Oct 24 20:23:31 2023 +0200

    Fix `@rails/ujs` imports

commit 4b0fb76
Merge: e25cc4d e93a75f
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Tue Oct 24 19:59:19 2023 +0200

    Merge commit 'e93a75f1a11d6dfdcbd39dbdc22526c5508ad881' into glitch-soc/merge-upstream

    Conflicts and ported changes:
    - updated `@rails/ujs` imports

commit 134de73
Author: Eugen Rochko <eugen@zeonfederated.com>
Date:   Tue Oct 24 19:45:08 2023 +0200

    Change icons in web UI (mastodon#27385)

    Co-authored-by: Renaud Chaput <renchap@gmail.com>

commit b188538
Author: Michael Stanclift <mx@vmstan.com>
Date:   Tue Oct 24 12:32:10 2023 -0500

    Fix missing libyaml-dev dependency in Dockerfile (mastodon#27533)

commit e25cc4d
Merge: 69ea596 379115e
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Tue Oct 24 19:31:14 2023 +0200

    Merge commit '379115e601361c2b5da775fbf28b7dff9dc02e71' into glitch-soc/merge-upstream

    Conflicts:
    - `config/navigation.rb`:
      Conflict due to glitch-soc having extra navigation items for its theming
      system.
      Ported upstream changes.

commit 69ea596
Author: Eugen Rochko <eugen@zeonfederated.com>
Date:   Mon Oct 23 17:07:13 2023 +0200

    [Glitch] Fix explore page reloading when you navigate back to it in web UI

    Port d9503a1 to glitch-soc

    Signed-off-by: Claire <claire.github-309c@sitedethib.com>

commit e6c3d3c
Merge: 79c154f d9503a1
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Tue Oct 24 19:21:01 2023 +0200

    Merge commit 'd9503a1965a16328b62ea7bf0bb54e87cbe30d29' into glitch-soc/merge-upstream

commit 79c154f
Merge: 57c0de9 fc7734d
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Tue Oct 24 19:20:14 2023 +0200

    Merge commit 'fc7734df8d8d53639bf244a2f2f3c39bc2fd12a7' into glitch-soc/merge-upstream

commit 57c0de9
Merge: 787d5ad 8b770ce
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Tue Oct 24 19:17:20 2023 +0200

    Merge commit '8b770ce8110e6cd609a6769c66210d95e291e3e5' into glitch-soc/merge-upstream

    Conflicts:
    - `config/initializers/content_security_policy.rb`:
      Upstream changed how asset host CSP directives are used, and glitch-soc has
      a pretty different CSP file.
      It may be worth reconsidering the differences between upstream and glitch-soc
      but for now, just port the change.

commit 787d5ad
Merge: 5323f0e bcae744
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Tue Oct 24 19:05:18 2023 +0200

    Merge commit 'bcae7442757845191c0e82b61cd1d0cd3ca860d1' into glitch-soc/merge-upstream

    Conflicts:
    - `yarn.lock`:
      Caused by a glitch-soc-only dependency (`exif-js`).
      Kept it.

commit 5323f0e
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Mon Oct 23 11:39:53 2023 +0200

    [Glitch] Fix incorrect proptypes from react-router-v5 update

    Port 0ad6617 to glitch-soc

    Signed-off-by: Claire <claire.github-309c@sitedethib.com>

commit 9325cb5
Author: Renaud Chaput <renchap@gmail.com>
Date:   Mon Oct 23 09:43:00 2023 +0200

    [Glitch] Convert `<Button>` to Typescript

    Port 9d45a44 to glitch-soc

    Signed-off-by: Claire <claire.github-309c@sitedethib.com>

commit 1138d44
Author: Eugen Rochko <eugen@zeonfederated.com>
Date:   Fri Oct 20 04:08:13 2023 +0200

    [Glitch] Fix missing background behind dismissable banner in web UI

    Port 33bd8ec to glitch-soc

    Signed-off-by: Claire <claire.github-309c@sitedethib.com>

commit b0c76ea
Merge: d6eacb7 0ad6617
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Tue Oct 24 18:41:20 2023 +0200

    Merge commit '0ad66175bf59a34b03d9ab2347181548d07089ea' into glitch-soc/merge-upstream

    Conflicts:
    - `app/javascript/mastodon/features/compose/components/compose_form.jsx`:
      Upstream changed one import, close to a glitch-soc-only import to handle
      different max character counts.
      Ported upstream's change.

commit 15182d1
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Tue Oct 24 15:24:57 2023 +0200

    Fix history handling not properly handling states after update to react-router v5 (mastodon#27526)

commit 6cf9f12
Author: Renaud Chaput <renchap@gmail.com>
Date:   Tue Oct 24 13:43:24 2023 +0200

    Create custom Github Actions for common steps (mastodon#27518)

commit 714e3ae
Author: Matt Jankowski <matt@jankowski.online>
Date:   Tue Oct 24 06:06:10 2023 -0400

    Use Rails 7.1 `normalizes` feature (mastodon#27521)

commit 50b7ea8
Author: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Date:   Tue Oct 24 11:51:34 2023 +0200

    New Crowdin Translations (automated) (mastodon#27523)

    Co-authored-by: GitHub Actions <noreply@github.com>

commit e923bb9
Author: Matt Jankowski <matt@jankowski.online>
Date:   Tue Oct 24 05:30:14 2023 -0400

    Use `ENV.fetch` in views, fixes rubocop haml-lint issue (mastodon#27527)

commit 4fdc548
Author: Matt Jankowski <matt@jankowski.online>
Date:   Tue Oct 24 04:08:31 2023 -0400

    Regenerate lint todos (mastodon#27513)

commit 9482810
Author: たいち ひ <taichi221228@icloud.com>
Date:   Tue Oct 24 17:06:14 2023 +0900

    Rewrite `emoji_unicode_mapping_light` to TS (mastodon#25444)

    Co-authored-by: taichi.fukuda ひ <taichi.fukuda@systemi.co.jp>

commit e93a75f
Author: Matt Jankowski <matt@jankowski.online>
Date:   Mon Oct 23 13:58:29 2023 -0400

    Rails 7.1 update (mastodon#25963)

commit 379115e
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Mon Oct 23 17:46:21 2023 +0200

    Add SELF_DESTRUCT env variable to process self-destructions in the background (mastodon#26439)

commit 26d2a2a
Author: Daniel M Brasil <danielmbrasil@protonmail.com>
Date:   Mon Oct 23 12:46:21 2023 -0300

    Migrate to request specs in `/api/v1/media` (mastodon#25543)

commit d9503a1
Author: Eugen Rochko <eugen@zeonfederated.com>
Date:   Mon Oct 23 17:07:13 2023 +0200

    Fix explore page reloading when you navigate back to it in web UI (mastodon#27489)

commit 79a6320
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Mon Oct 23 16:21:48 2023 +0200

    Fix Layout/EmptyLineAfterGuardClause issues caused by merging an old PR (mastodon#27512)

commit fc7734d
Author: Renaud Chaput <renchap@gmail.com>
Date:   Mon Oct 23 14:38:36 2023 +0200

    Remove Renovate's Lock File Maintenance task (mastodon#27510)

commit 3cfdab6
Author: Matt Jankowski <matt@jankowski.online>
Date:   Mon Oct 23 08:35:56 2023 -0400

    Prune the changelog of older entries (mastodon#27432)

commit d5408db
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Mon Oct 23 14:33:17 2023 +0200

    Update dependency prom-client to v15 (mastodon#27330)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit 8b770ce
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Mon Oct 23 14:30:46 2023 +0200

    Add warnings to report action logs (mastodon#27425)

commit 44edf3a
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Mon Oct 23 14:28:32 2023 +0200

    Improve error handling in mastodon:setup task (mastodon#21464)

commit 99f2534
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Mon Oct 23 14:27:57 2023 +0200

    Add support for displaying unknown servers on /admin/instances/:domain (mastodon#27150)

commit c3e0eb3
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Mon Oct 23 14:27:07 2023 +0200

    Change Content-Security-Policy to be tighter on media paths (mastodon#26889)

commit bcae744
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Mon Oct 23 14:19:38 2023 +0200

    Fix some link anchors being recognized as hashtags (mastodon#27271)

commit 53fd28b
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Mon Oct 23 12:45:14 2023 +0200

    Update dependency react-select to v5.7.7 (mastodon#27003)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit f7b3470
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Mon Oct 23 12:36:47 2023 +0200

    Update babel monorepo to v7.23.2 (mastodon#27118)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit 25751c6
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Mon Oct 23 12:35:54 2023 +0200

    Update DefinitelyTyped types (non-major) (mastodon#27414)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit 1322040
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Mon Oct 23 12:20:01 2023 +0200

    Update dependency core-js to v3.33.1 (mastodon#27481)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit bb5bae1
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Mon Oct 23 12:18:16 2023 +0200

    Update dependency webpack-merge to v5.10.0 (mastodon#27420)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit 69d3318
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Mon Oct 23 12:16:02 2023 +0200

    Update dependency rimraf to v5.0.5 (mastodon#27140)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit cd08b43
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Mon Oct 23 12:15:31 2023 +0200

    Update dependency sass to v1.69.4 (mastodon#27004)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit 6f1eb89
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Mon Oct 23 12:14:16 2023 +0200

    Update dependency ws to v8.14.2 (mastodon#26980)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit 6f429d3
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Mon Oct 23 12:13:12 2023 +0200

    Update formatjs monorepo (mastodon#27416)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit 773133b
Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Date:   Mon Oct 23 12:12:17 2023 +0200

    Update eslint (non-major) (mastodon#27415)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

commit 0ad6617
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Mon Oct 23 11:39:53 2023 +0200

    Fix incorrect proptypes from react-router-v5 update (mastodon#27507)

commit 70fd819
Author: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Date:   Mon Oct 23 10:04:54 2023 +0200

    New Crowdin Translations (automated) (mastodon#27490)

    Co-authored-by: GitHub Actions <noreply@github.com>

commit b021347
Author: Matt Jankowski <matt@jankowski.online>
Date:   Mon Oct 23 03:50:02 2023 -0400

    Validate allowed schemes on preview card URLs (mastodon#27485)

commit 9d45a44
Author: Renaud Chaput <renchap@gmail.com>
Date:   Mon Oct 23 09:43:00 2023 +0200

    Convert `<Button>` to Typescript (mastodon#27492)

commit d6eacb7
Author: Renaud Chaput <renchap@gmail.com>
Date:   Thu Oct 19 19:44:55 2023 +0200

    [Glitch] Upgrade to react-router v5

    Port 1b70d7e to glitch-soc

    Co-authored-by: Claire <claire.github-309c@sitedethib.com>
    Signed-off-by: Claire <claire.github-309c@sitedethib.com>

commit 245513d
Merge: 18eacc7 1b70d7e
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Sat Oct 21 14:33:33 2023 +0200

    Merge commit '1b70d7ed7c0fd3a9fcf028bf76b8c62ac8b3897f' into glitch-soc/merge-upstream

commit ab0fb81
Author: Matt Jankowski <matt@jankowski.online>
Date:   Fri Oct 20 09:32:16 2023 -0400

    Configure brakeman to ignore url safe preview card urls (mastodon#25883)

commit 1368853
Author: Claire <claire.github-309c@sitedethib.com>
Date:   Fri Oct 20 10:45:46 2023 +0200

    Fix processing LDSigned activities from actors with unknown public keys (mastodon#27474)

commit 284c719
Author: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Date:   Fri Oct 20 10:11:40 2023 +0200

    New Crowdin Translations (automated) (mastodon#27480)

    Co-authored-by: GitHub Actions <noreply@github.com>

commit 33bd8ec
Author: Eugen Rochko <eugen@zeonfederated.com>
Date:   Fri Oct 20 04:08:13 2023 +0200

    Fix missing background behind dismissable banner in web UI (mastodon#27479)

commit 1b70d7e
Author: Renaud Chaput <renchap@gmail.com>
Date:   Thu Oct 19 19:44:55 2023 +0200

    Upgrade to react-router v5 (mastodon#25047)

    Co-authored-by: Claire <claire.github-309c@sitedethib.com>
@Andre601
Copy link

Andre601 commented Dec 4, 2023

I just say it straight away: These new reply icons are ugly... a simple line without any character whatsoever...

Why was this icon set selected? I personally would've recommended @mdi/svg which offers icons that look much more similar to the fontawesome icons... In addition are they also open source and free.

The current ones just feel... off.

And just to give a point is here a quick comparison

  • Font Awesome (v5 pulled from iconify.design)
    grafik
  • New Icons used
    grafik
  • @mdi/svg icons
    grafik

The mid ones have much more character and aren't as thin as the ones you now want to use, which at least for me are even hard to see when used (I'm on an instance that uses a dev build including these icons).

Also, the animation when adding a favourite seems not to be centered on the icon, having it spin with a off-center position...

000hen pushed a commit to thenapnetwork/nap-mastodon that referenced this pull request Dec 6, 2023
Co-authored-by: Renaud Chaput <renchap@gmail.com>
vmstan pushed a commit to vmstan/mastodon that referenced this pull request Dec 14, 2023
Co-authored-by: Renaud Chaput <renchap@gmail.com>
vmstan pushed a commit to vmstan/mastodon that referenced this pull request Jan 5, 2024
Co-authored-by: Renaud Chaput <renchap@gmail.com>
@brendanjones
Copy link

brendanjones commented Feb 19, 2024

I realise this is merged already, but wanted to ask why the Explore menu item doesn't use the Material explore icon (a compass)?

  1. It is the 'correct' icon for such a menu item (at least on Android).
  2. The Explore menu doesn't only let you explore hashtags, so using a hashtag icon excludes representing exploring posts, people and news.
  3. Not using the hashtag icon would free up that icon to be used for "pin" hashtags in simple layout #11091

I know Twitter used the hashtag icon for explore so people are familiar with it, but we don't need to always follow what Twitter does, do we? :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ui Front-end, design
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change Icon Library Consider: SVG for icons?