Merge pull request #3140 from LucasXu0/url_protocol_in_windows_installer

frontend/appflowy_flutter/lib/user/presentation/sign_in_screen.dart

@@ -71,6 +71,8 @@ class SignInForm extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
+    final isSubmitting = context.read<SignInBloc>().state.isSubmitting;
+    const indicatorMinHeight = 4.0;
     return Align(
       alignment: Alignment.center,
       child: AuthFormContainer(
@@ -104,13 +106,19 @@ class SignInForm extends StatelessWidget {
           const VSpace(10),
           const ThirdPartySignInButtons(),
           const VSpace(20),
-
           // loading status
-          if (context.read<SignInBloc>().state.isSubmitting) ...[
+          ...isSubmitting
-            const SizedBox(height: 8),
+              ? [
-            const LinearProgressIndicator(value: null),
+                  const VSpace(indicatorMinHeight),
-            const VSpace(20),
+                  const LinearProgressIndicator(
-          ],
+                    value: null,
+                    minHeight: indicatorMinHeight,
+                  ),
+                ]
+              : [
+                  const VSpace(indicatorMinHeight * 2.0)
+                ], // add the same space when there's no loading status.
+          const VSpace(20)
         ],
       ),
     );

frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/folder/personal_folder.dart

@@ -90,6 +90,7 @@ class _PersonalFolderHeaderState extends State<PersonalFolderHeader> {
   @override
   Widget build(BuildContext context) {
     const iconSize = 26.0;
+    const textPadding = 4.0;
     return MouseRegion(
       onEnter: (event) => setState(() => onHover = true),
       onExit: (event) => setState(() => onHover = false),
@@ -99,8 +100,10 @@ class _PersonalFolderHeaderState extends State<PersonalFolderHeader> {
           FlowyTextButton(
             LocaleKeys.sideBar_personal.tr(),
             tooltip: LocaleKeys.sideBar_clickToHidePersonal.tr(),
-            constraints: const BoxConstraints(maxHeight: iconSize),
+            constraints: const BoxConstraints(
-            padding: const EdgeInsets.all(4),
+              minHeight: iconSize + textPadding * 2,
+            ),
+            padding: const EdgeInsets.all(textPadding),
             fillColor: Colors.transparent,
             onPressed: widget.onPressed,
           ),

frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/settings_user_view.dart

@@ -30,6 +30,7 @@ class SettingsUserView extends StatelessWidget {
   // Called when the user open a historical user in the setting dialog
   final VoidCallback didOpenUser;
   final UserProfilePB user;
+
   SettingsUserView(
     this.user, {
     required this.didLogin,
@@ -44,7 +45,8 @@ class SettingsUserView extends StatelessWidget {
       create: (context) => getIt<SettingsUserViewBloc>(param1: user)
         ..add(const SettingsUserEvent.initial()),
       child: BlocBuilder<SettingsUserViewBloc, SettingsUserState>(
-        builder: (context, state) => Column(
+        builder: (context, state) => SingleChildScrollView(
+          child: Column(
             crossAxisAlignment: CrossAxisAlignment.start,
             mainAxisSize: MainAxisSize.min,
             children: [
@@ -55,12 +57,12 @@ class SettingsUserView extends StatelessWidget {
               _renderCurrentOpenaiKey(context),
               const VSpace(20),
               _renderHistoricalUser(context),
-            const Spacer(),
               _renderLoginOrLogoutButton(context, state),
               const VSpace(20),
             ],
           ),
         ),
+      ),
     );
   }
 
@@ -107,10 +109,13 @@ class SettingsUserView extends StatelessWidget {
   }
 
   Widget _renderLogoutButton(BuildContext context) {
-    return FlowyButton(
+    return Tooltip(
-      useIntrinsicWidth: true,
+      message: LocaleKeys.settings_user_clickToLogout.tr(),
-      text: FlowyText(
+      child: FlowyButton(
+        margin: const EdgeInsets.symmetric(vertical: 8.0, horizontal: 2.0),
+        text: FlowyText.medium(
           LocaleKeys.settings_menu_logout.tr(),
+          fontSize: 13,
         ),
         onTap: () async {
           NavigatorAlertDialog(
@@ -121,6 +126,7 @@ class SettingsUserView extends StatelessWidget {
             },
           ).show(context);
         },
+      ),
     );
   }
 

frontend/resources/translations/en.json

@@ -296,7 +296,8 @@
       "name": "Name",
       "icon": "Icon",
       "selectAnIcon": "Select an icon",
-      "pleaseInputYourOpenAIKey": "please input your OpenAI key"
+      "pleaseInputYourOpenAIKey": "please input your OpenAI key",
+      "clickToLogout": "Click to logout the current user"
     },
     "shortcuts": {
       "shortcutsLabel": "Shortcuts",

frontend/scripts/windows_installer/inno_setup_config.iss

@@ -21,3 +21,9 @@ Source: "AppFlowy\data\*";DestDir: "{app}\data\"; Flags: recursesubdirs
 [Icons]
 Name: "{userdesktop}\AppFlowy"; Filename: "{app}\AppFlowy.exe"
 Name: "{group}\AppFlowy"; Filename: "{app}\AppFlowy.exe"
+
+[Registry]
+Root: HKCR; Subkey: "AppFlowy"; ValueType: "string"; ValueData: "URL:Custom Protocol"; Flags: uninsdeletekey
+Root: HKCR; Subkey: "AppFlowy"; ValueType: "string"; ValueName: "URL Protocol"; ValueData: ""
+Root: HKCR; Subkey: "AppFlowy\DefaultIcon"; ValueType: "string"; ValueData: "{app}\AppFlowy.exe,0"
+Root: HKCR; Subkey: "AppFlowy\shell\open\command"; ValueType: "string"; ValueData: """{app}\AppFlowy.exe"" ""%1"""