From 38142182774ea772aacc88f26586512d6279267f Mon Sep 17 00:00:00 2001 From: Henry Jameson Date: Mon, 19 Oct 2020 19:38:49 +0300 Subject: Some initial work on replacing icons with FA5 --- src/components/reply_button/reply_button.vue | 28 ++++++++++++++++++++++++---- 1 file changed, 24 insertions(+), 4 deletions(-) (limited to 'src/components/reply_button/reply_button.vue') diff --git a/src/components/reply_button/reply_button.vue b/src/components/reply_button/reply_button.vue index b2904b5c..ae7b0e26 100644 --- a/src/components/reply_button/reply_button.vue +++ b/src/components/reply_button/reply_button.vue @@ -1,15 +1,19 @@ + + -- cgit v1.2.3-70-g09d2 From b6cf2bcefd5ddd20c11ee4b0d0f94e29e8f59c40 Mon Sep 17 00:00:00 2001 From: Henry Jameson Date: Wed, 28 Oct 2020 21:31:16 +0200 Subject: improved the semantics for our icon scale, fixed preview, fixed navbar desktop --- src/App.scss | 12 ++++--- src/App.vue | 9 ++--- src/components/chat_new/chat_new.vue | 3 +- src/components/extra_buttons/extra_buttons.vue | 2 +- src/components/favorite_button/favorite_button.vue | 6 ++-- .../global_notice_list/global_notice_list.vue | 2 +- src/components/image_cropper/image_cropper.vue | 2 +- src/components/login_form/login_form.vue | 2 +- src/components/mfa_form/recovery_form.vue | 2 +- src/components/mfa_form/totp_form.vue | 2 +- src/components/mobile_nav/mobile_nav.vue | 9 ++--- src/components/nav_panel/nav_panel.vue | 41 ++++++++-------------- src/components/notification/notification.vue | 8 ++--- src/components/password_reset/password_reset.vue | 2 +- .../post_status_form/post_status_form.vue | 16 ++++----- src/components/react_button/react_button.vue | 3 +- src/components/reply_button/reply_button.vue | 6 ++-- src/components/retweet_button/retweet_button.vue | 9 ++--- src/components/scope_selector/scope_selector.vue | 12 +++---- src/components/search_bar/search_bar.vue | 6 ++-- src/components/settings_modal/tabs/profile_tab.vue | 4 +-- .../settings_modal/tabs/theme_tab/preview.vue | 8 ++--- src/components/side_drawer/side_drawer.vue | 38 +++++++------------- src/components/status/status.vue | 24 +++++-------- src/components/timeline_menu/timeline_menu.vue | 15 +++----- src/components/user_card/user_card.vue | 3 +- 26 files changed, 95 insertions(+), 151 deletions(-) (limited to 'src/components/reply_button/reply_button.vue') diff --git a/src/App.scss b/src/App.scss index 06915e16..6884f314 100644 --- a/src/App.scss +++ b/src/App.scss @@ -812,14 +812,18 @@ nav { } } -.button-icon { - &.svg-inline--fa.fa-lg { - display: inline-block; - padding: 0 0.3em; +.fa-scale-110 { + &.svg-inline--fa { font-size: 1.1em; } } +.fa-old-padding { + &.svg-inline--fa { + padding: 0 0.3em; + } +} + @keyframes shakeError { 0% { transform: translateX(0); diff --git a/src/App.vue b/src/App.vue index c27f51bf..6e44c7e9 100644 --- a/src/App.vue +++ b/src/App.vue @@ -53,8 +53,7 @@ > @@ -66,8 +65,7 @@ target="_blank" > @@ -78,8 +76,7 @@ @click.prevent="logout" > diff --git a/src/components/chat_new/chat_new.vue b/src/components/chat_new/chat_new.vue index 8f02a699..f3894a3a 100644 --- a/src/components/chat_new/chat_new.vue +++ b/src/components/chat_new/chat_new.vue @@ -20,8 +20,7 @@
diff --git a/src/components/extra_buttons/extra_buttons.vue b/src/components/extra_buttons/extra_buttons.vue index 4bf1e62a..f32ea02d 100644 --- a/src/components/extra_buttons/extra_buttons.vue +++ b/src/components/extra_buttons/extra_buttons.vue @@ -107,7 +107,7 @@
diff --git a/src/components/favorite_button/favorite_button.vue b/src/components/favorite_button/favorite_button.vue index 6c7bfdab..dfe12f86 100644 --- a/src/components/favorite_button/favorite_button.vue +++ b/src/components/favorite_button/favorite_button.vue @@ -2,11 +2,10 @@
{{ status.fave_num }} @@ -14,10 +13,9 @@
{{ status.fave_num }}
diff --git a/src/components/global_notice_list/global_notice_list.vue b/src/components/global_notice_list/global_notice_list.vue index 06b6af9c..8a33b9eb 100644 --- a/src/components/global_notice_list/global_notice_list.vue +++ b/src/components/global_notice_list/global_notice_list.vue @@ -10,7 +10,7 @@ {{ $t(notice.messageKey, notice.messageArgs) }}
diff --git a/src/components/image_cropper/image_cropper.vue b/src/components/image_cropper/image_cropper.vue index 605f7427..75def612 100644 --- a/src/components/image_cropper/image_cropper.vue +++ b/src/components/image_cropper/image_cropper.vue @@ -43,7 +43,7 @@ > {{ submitErrorMsg }} diff --git a/src/components/login_form/login_form.vue b/src/components/login_form/login_form.vue index 4ab5d192..a1f77210 100644 --- a/src/components/login_form/login_form.vue +++ b/src/components/login_form/login_form.vue @@ -77,7 +77,7 @@
{{ error }} diff --git a/src/components/mfa_form/recovery_form.vue b/src/components/mfa_form/recovery_form.vue index f6526d2a..78953649 100644 --- a/src/components/mfa_form/recovery_form.vue +++ b/src/components/mfa_form/recovery_form.vue @@ -55,7 +55,7 @@
{{ error }} diff --git a/src/components/mfa_form/totp_form.vue b/src/components/mfa_form/totp_form.vue index 0c6412ea..9401cad5 100644 --- a/src/components/mfa_form/totp_form.vue +++ b/src/components/mfa_form/totp_form.vue @@ -58,7 +58,7 @@ {{ error }} diff --git a/src/components/mobile_nav/mobile_nav.vue b/src/components/mobile_nav/mobile_nav.vue index db9b0ba0..d2bc65ab 100644 --- a/src/components/mobile_nav/mobile_nav.vue +++ b/src/components/mobile_nav/mobile_nav.vue @@ -16,8 +16,7 @@ @click.stop.prevent="toggleMobileSidebar()" >
diff --git a/src/components/nav_panel/nav_panel.vue b/src/components/nav_panel/nav_panel.vue index 4755d7b7..e7301328 100644 --- a/src/components/nav_panel/nav_panel.vue +++ b/src/components/nav_panel/nav_panel.vue @@ -9,22 +9,18 @@ > - {{ $t("nav.timelines") }} + />{{ $t("nav.timelines") }}
  • - {{ $t("nav.interactions") }} + />{{ $t("nav.interactions") }}
  • @@ -37,22 +33,18 @@
  • - {{ $t("nav.chats") }} + />{{ $t("nav.chats") }}
  • - {{ $t("nav.friend_requests") }} + />{{ $t("nav.friend_requests") }} @@ -94,7 +85,7 @@ } .follow-request-count { - margin: -6px 10px; + vertical-align: bottom; background-color: $fallback--bg; background-color: var(--input, $fallback--faint); } @@ -126,7 +117,8 @@ a { display: block; - padding: 0.8em 0.85em; + align-items: stretch; + padding: 0.9em 1em; &:hover { background-color: $fallback--lightBg; @@ -156,13 +148,8 @@ } } - .button-icon { - margin-left: -0.1em; - margin-right: 0.2em; - } - - .button-icon:before { - width: 1.1em; + .fa-scale-110 { + margin-right: 0.8em; } } diff --git a/src/components/notification/notification.vue b/src/components/notification/notification.vue index b609ae04..2bbde108 100644 --- a/src/components/notification/notification.vue +++ b/src/components/notification/notification.vue @@ -19,7 +19,7 @@ class="unmute" @click.prevent="toggleMute" >
  • @@ -137,7 +137,7 @@ href="#" @click.prevent="toggleMute" > @@ -157,13 +157,13 @@ >
    diff --git a/src/components/reply_button/reply_button.vue b/src/components/reply_button/reply_button.vue index ae7b0e26..a0ac8941 100644 --- a/src/components/reply_button/reply_button.vue +++ b/src/components/reply_button/reply_button.vue @@ -2,9 +2,8 @@
    diff --git a/src/components/retweet_button/retweet_button.vue b/src/components/retweet_button/retweet_button.vue index 706c6b81..b234f3d9 100644 --- a/src/components/retweet_button/retweet_button.vue +++ b/src/components/retweet_button/retweet_button.vue @@ -3,9 +3,8 @@ @@ -25,9 +23,8 @@
    {{ status.repeat_num }} diff --git a/src/components/scope_selector/scope_selector.vue b/src/components/scope_selector/scope_selector.vue index a43af23b..a22a4fda 100644 --- a/src/components/scope_selector/scope_selector.vue +++ b/src/components/scope_selector/scope_selector.vue @@ -12,8 +12,7 @@ >
    diff --git a/src/components/search_bar/search_bar.vue b/src/components/search_bar/search_bar.vue index 442e91b2..f1c3fd7a 100644 --- a/src/components/search_bar/search_bar.vue +++ b/src/components/search_bar/search_bar.vue @@ -8,8 +8,7 @@ :title="$t('nav.search')" > @@ -34,10 +33,9 @@ diff --git a/src/components/settings_modal/tabs/profile_tab.vue b/src/components/settings_modal/tabs/profile_tab.vue index 5a659fc8..d62bc392 100644 --- a/src/components/settings_modal/tabs/profile_tab.vue +++ b/src/components/settings_modal/tabs/profile_tab.vue @@ -235,7 +235,7 @@ > Error: {{ bannerUploadError }} @@ -286,7 +286,7 @@ Error: {{ backgroundUploadError }} diff --git a/src/components/settings_modal/tabs/theme_tab/preview.vue b/src/components/settings_modal/tabs/theme_tab/preview.vue index 20201e18..3174fd7c 100644 --- a/src/components/settings_modal/tabs/theme_tab/preview.vue +++ b/src/components/settings_modal/tabs/theme_tab/preview.vue @@ -42,25 +42,25 @@
    diff --git a/src/components/side_drawer/side_drawer.vue b/src/components/side_drawer/side_drawer.vue index 2a4d794a..eac0ddb2 100644 --- a/src/components/side_drawer/side_drawer.vue +++ b/src/components/side_drawer/side_drawer.vue @@ -37,9 +37,8 @@ > {{ $t("login.login") }} @@ -50,9 +49,8 @@ > {{ $t("nav.timelines") }} @@ -66,9 +64,8 @@ style="position: relative" > {{ $t("nav.chats") }} {{ $t("nav.interactions") }} @@ -97,9 +93,8 @@ > {{ $t("nav.friend_requests") }} {{ $t("shoutbox.title") }} @@ -131,9 +125,8 @@ > {{ $t("nav.search") }} @@ -144,9 +137,8 @@ > {{ $t("nav.who_to_follow") }} @@ -157,9 +149,8 @@ @click="openSettingsModal" > {{ $t("settings.settings") }} @@ -167,9 +158,8 @@
  • {{ $t("nav.about") }} @@ -183,9 +173,8 @@ target="_blank" > {{ $t("nav.administration") }} @@ -199,9 +188,8 @@ @click="doLogout" > {{ $t("login.logout") }} @@ -284,7 +272,7 @@ --lightText: var(--popoverLightText, $fallback--lightText); --icon: var(--popoverIcon, $fallback--icon); - .button-icon:before { + .fa-scale-110 fa-old-padding:before { width: 1.1em; } } diff --git a/src/components/status/status.vue b/src/components/status/status.vue index c94862d4..21412faa 100644 --- a/src/components/status/status.vue +++ b/src/components/status/status.vue @@ -11,7 +11,7 @@ > {{ error }} @@ -22,7 +22,7 @@ @@ -49,13 +49,12 @@
  • @@ -186,9 +185,8 @@ :title="status.visibility | capitalize" >
    @@ -249,9 +244,8 @@ @click.prevent="gotoOriginal(status.in_reply_to_status_id)" > {{ $t("nav.timeline") }} @@ -28,8 +27,7 @@ {{ $t("nav.bookmarks") }} @@ -38,8 +36,7 @@ {{ $t("nav.dms") }} @@ -48,8 +45,7 @@ {{ $t("nav.public_tl") }} @@ -58,8 +54,7 @@ {{ $t("nav.twkn") }} diff --git a/src/components/user_card/user_card.vue b/src/components/user_card/user_card.vue index c5f10b75..55e231d2 100644 --- a/src/components/user_card/user_card.vue +++ b/src/components/user_card/user_card.vue @@ -22,9 +22,8 @@ /> -- cgit v1.2.3-70-g09d2 From 7b99d98c553f40ec4d633d0d4fdf65f275c80e77 Mon Sep 17 00:00:00 2001 From: Shpuld Shpuldson Date: Tue, 24 Nov 2020 12:32:42 +0200 Subject: Replace all use of + href='#' with proper buttons --- src/App.scss | 32 +++++++++++-- src/components/account_actions/account_actions.vue | 14 +++--- .../async_component_error.vue | 2 +- src/components/attachment/attachment.js | 6 ++- src/components/attachment/attachment.vue | 28 ++++++----- src/components/block_card/block_card.vue | 4 +- src/components/chat_list/chat_list.vue | 5 +- src/components/chat_message/chat_message.scss | 3 -- src/components/chat_message/chat_message.vue | 4 +- src/components/conversation/conversation.vue | 13 +++--- src/components/desktop_nav/desktop_nav.scss | 10 ++-- src/components/desktop_nav/desktop_nav.vue | 40 ++++++++-------- .../domain_mute_card/domain_mute_card.vue | 4 +- src/components/emoji_reactions/emoji_reactions.vue | 2 +- src/components/export_import/export_import.vue | 4 +- src/components/exporter/exporter.vue | 2 +- src/components/extra_buttons/extra_buttons.vue | 28 +++++------ src/components/favorite_button/favorite_button.js | 5 -- src/components/favorite_button/favorite_button.vue | 38 +++++++-------- src/components/follow_button/follow_button.vue | 2 +- .../follow_request_card/follow_request_card.vue | 4 +- src/components/image_cropper/image_cropper.vue | 6 +-- src/components/importer/importer.vue | 2 +- src/components/login_form/login_form.vue | 2 +- src/components/media_upload/media_upload.vue | 54 +++++++++------------- src/components/mfa_form/recovery_form.vue | 14 +++--- src/components/mfa_form/totp_form.vue | 14 +++--- src/components/mobile_nav/mobile_nav.vue | 14 +++--- .../mobile_post_status_button.vue | 2 +- .../moderation_tools/moderation_tools.vue | 6 +-- src/components/mute_card/mute_card.vue | 4 +- src/components/notification/notification.vue | 13 +++--- src/components/notifications/notifications.vue | 8 ++-- src/components/password_reset/password_reset.vue | 2 +- src/components/poll/poll.vue | 2 +- .../post_status_form/post_status_form.vue | 34 ++++++-------- src/components/react_button/react_button.vue | 13 ++++-- src/components/registration/registration.vue | 2 +- src/components/reply_button/reply_button.vue | 19 ++++---- src/components/retweet_button/retweet_button.js | 5 -- src/components/retweet_button/retweet_button.vue | 48 +++++++++---------- src/components/search_bar/search_bar.vue | 32 +++++++------ src/components/settings_modal/settings_modal.vue | 4 +- .../settings_modal/tabs/mutes_and_blocks_tab.vue | 10 ++-- .../settings_modal/tabs/notifications_tab.vue | 2 +- src/components/settings_modal/tabs/profile_tab.vue | 8 ++-- .../settings_modal/tabs/security_tab/confirm.vue | 4 +- .../settings_modal/tabs/security_tab/mfa.vue | 10 ++-- .../settings_modal/tabs/security_tab/mfa_totp.vue | 4 +- .../tabs/security_tab/security_tab.vue | 10 ++-- .../settings_modal/tabs/theme_tab/preview.vue | 4 +- .../settings_modal/tabs/theme_tab/theme_tab.vue | 28 +++++------ src/components/shadow_control/shadow_control.vue | 8 ++-- src/components/side_drawer/side_drawer.vue | 15 +++--- src/components/status/status.vue | 35 +++++++------- src/components/status_content/status_content.vue | 37 +++++++-------- src/components/tab_switcher/tab_switcher.js | 2 +- src/components/timeline/timeline.vue | 8 ++-- src/components/user_card/user_card.vue | 10 ++-- .../user_reporting_modal/user_reporting_modal.vue | 2 +- 60 files changed, 384 insertions(+), 363 deletions(-) (limited to 'src/components/reply_button/reply_button.vue') diff --git a/src/App.scss b/src/App.scss index ca7d33cd..ef2a13b1 100644 --- a/src/App.scss +++ b/src/App.scss @@ -33,6 +33,7 @@ h4 { max-width: 980px; align-content: flex-start; } + .underlay { background-color: rgba(0,0,0,0.15); background-color: var(--underlay, rgba(0,0,0,0.15)); @@ -69,7 +70,7 @@ a { color: var(--link, $fallback--link); } -button { +.button-default { user-select: none; color: $fallback--text; color: var(--btnText, $fallback--text); @@ -85,7 +86,9 @@ button { font-family: sans-serif; font-family: var(--interfaceFont, sans-serif); - i[class*=icon-], .svg-inline--fa { + i[class*=icon-], + :not(&.-icon), + .svg-inline--fa { color: $fallback--text; color: var(--btnText, $fallback--text); } @@ -149,6 +152,29 @@ button { } } +.button-unstyled { + background: none; + border: none; + outline: none; + display: inline; + text-align: initial; + font-size: 100%; + font-family: inherit; + padding: 0; + line-height: unset; + cursor: pointer; + + &.-link { + color: $fallback--link; + color: var(--link, $fallback--link); + } + + &.-padded { + padding: 5px; + margin: -5px; + } +} + input, textarea, .select, .input { &.unstyled { @@ -797,7 +823,7 @@ nav { } } -.btn.btn-default { +.btn.button-default { min-height: 28px; } diff --git a/src/components/account_actions/account_actions.vue b/src/components/account_actions/account_actions.vue index e3ae376e..c10b09b8 100644 --- a/src/components/account_actions/account_actions.vue +++ b/src/components/account_actions/account_actions.vue @@ -13,14 +13,14 @@ diff --git a/src/components/chat_message/chat_message.scss b/src/components/chat_message/chat_message.scss index 5af744a3..e4351d3b 100644 --- a/src/components/chat_message/chat_message.scss +++ b/src/components/chat_message/chat_message.scss @@ -31,9 +31,6 @@ color: $fallback--text; color: var(--text, $fallback--text); } - - border-radius: $fallback--chatMessageRadius; - border-radius: var(--chatMessageRadius, $fallback--chatMessageRadius); } .popover { diff --git a/src/components/chat_message/chat_message.vue b/src/components/chat_message/chat_message.vue index 3849ab6e..0777f880 100644 --- a/src/components/chat_message/chat_message.vue +++ b/src/components/chat_message/chat_message.vue @@ -53,7 +53,7 @@ - - + - + + + diff --git a/src/components/domain_mute_card/domain_mute_card.vue b/src/components/domain_mute_card/domain_mute_card.vue index 97aee243..3b5aec14 100644 --- a/src/components/domain_mute_card/domain_mute_card.vue +++ b/src/components/domain_mute_card/domain_mute_card.vue @@ -6,7 +6,7 @@ {{ $t('domain_mute_card.unmute') }}