feat: star rating and sort for extension manager

Author: abose

src/extensibility/ExtensionManagerView.js

@@ -298,12 +298,17 @@ define(function (require, exports, module) {
             isVerified = true;
         }
 
+        if(context.gihubStars){
+            context.githubStarCount = context.gihubStars;
+        }
+
         if (entry.registryInfo) {
             var latestVerCompatInfo = ExtensionManager.getCompatibilityInfo(entry.registryInfo, brackets.metadata.apiVersion);
             context.isCompatible = latestVerCompatInfo.isCompatible;
             context.requiresNewer = latestVerCompatInfo.requiresNewer;
             context.isCompatibleLatest = latestVerCompatInfo.isLatestVersion;
             entry.registryInfo.isVerified = isVerified ? 1 : 0;
+            entry.registryInfo.githubStarCount = context.githubStarCount || 0;
             if (!context.isCompatibleLatest) {
                 var installWarningBase = context.requiresNewer ? Strings.EXTENSION_LATEST_INCOMPATIBLE_NEWER : Strings.EXTENSION_LATEST_INCOMPATIBLE_OLDER;
                 context.installWarning = StringUtils.format(installWarningBase, entry.registryInfo.versions[entry.registryInfo.versions.length - 1].version, latestVerCompatInfo.compatibleVersion);

src/extensibility/registry_utils.js

@@ -106,7 +106,8 @@ define(function (require, exports, module) {
             result = htmlEscape(userId);
         }
         if (ownerLink) {
-            result = "<a href='" + htmlEscape(ownerLink) + "' title='" + htmlEscape(ownerLink) + "'>" + result + "</a>";
+            result = `<a class="no-active-border" href='`
+                + htmlEscape(ownerLink) + "' title='" + htmlEscape(ownerLink) + "'>" + result + "</a>";
         }
         return result;
     };

src/htmlContent/extension-manager-dialog.html

@@ -28,6 +28,7 @@
                <select class="sort-extensions">
                    <option value="publishedDate">{{Strings.EXTENSIONS_LAST_UPDATED}}</option>
                    <option value="totalDownloads">{{Strings.EXTENSIONS_DOWNLOADS}}</option>
+                   <option value="githubStarCount">{{Strings.EXTENSION_STAR_SORT}}</option>
                    <option value="isVerified">{{Strings.EXTENSION_VERIFIED_SORT}}</option>
                </select>
             </div>

src/htmlContent/extension-manager-view-item.html

@@ -5,7 +5,7 @@
             <span class="muted ext-author">
                 {{{authorInfo}}}
                 {{#verified}}
-                <a href="{{verifiedURL}}">
+                <a href="{{verifiedURL}}" class="no-active-border">
                     <img class="extn-verified" title="{{verified}}" src="styles/images/verified.svg" style="width:1.5em;">
                 </a>
                 {{/verified}}
@@ -24,15 +24,17 @@
                 {{/downloadCount}}
             </div>
             <div class="{{^githubStarCount}} forced-inVisible {{/githubStarCount}}">
-                <div class="github-stars-button">
-                    <div class="parent">
+                <a href="{{ownerRepo}}" class="no-active-border">
+                    <div class="github-stars-button">
+                        <div class="parent">
                         <span class="starContainer">
                             <i class="fa-regular fa-star star"></i>
-                            <span>Star</span>
+                            <span>{{Strings.EXTENSION_STAR}}</span>
                         </span>
-                        <span class="count"> {{githubStarCount}} </span>
+                            <span class="count"> {{githubStarCount}} </span>
+                        </div>
                     </div>
-                </div>
+                </a>
             </div>
         </div>
     </td>

src/nls/root/strings.js

@@ -591,6 +591,8 @@ define({
     "UNKNOWN_ERROR": "Unknown internal error.",
     "EXTENSION_VERIFIED_PUBLISHER": "This publisher has verified ownership of {0}",
     "EXTENSION_VERIFIED_SORT": "Verified",
+    "EXTENSION_STAR": "Star",
+    "EXTENSION_STAR_SORT": "Star Rating",
     // For NOT_FOUND_ERR, see generic strings above
     "EXTENSION_MANAGER_TITLE": "Extension Manager",
     "EXTENSION_MANAGER_ERROR_LOAD": "Unable to access the extension registry. Please try again later.",

src/styles/brackets_patterns_override.less

@@ -1161,6 +1161,11 @@ a[href^="http"] {
     background-color: @bc-panel-bg-promoted;
     min-width: @extension-manager-min-width;
 
+    .no-active-border:focus{
+        border-color: transparent !important;
+        box-shadow: none !important;
+    }
+
     .current-theme-apply-hide{
         display: none;
     }