فهرست منبع

Show the last tag (if available) as expanded by default instead of dev-master

Jordi Boggiano 10 سال پیش
والد
کامیت
64aeb9a183

+ 9 - 0
src/Packagist/WebBundle/Controller/WebController.php

@@ -653,10 +653,19 @@ class WebController extends Controller
             $version = $versionRepo->getFullVersion(reset($versions)->getId());
             $version = $versionRepo->getFullVersion(reset($versions)->getId());
         }
         }
 
 
+        $expandedVersion = reset($versions);
+        foreach ($versions as $v) {
+            if (!$v->isDevelopment()) {
+                $expandedVersion = $v;
+                break;
+            }
+        }
+
         $data = array(
         $data = array(
             'package' => $package,
             'package' => $package,
             'version' => $version,
             'version' => $version,
             'versions' => $versions,
             'versions' => $versions,
+            'expandedVersion' => $expandedVersion,
         );
         );
 
 
         try {
         try {

+ 4 - 3
src/Packagist/WebBundle/Resources/views/Web/viewPackage.html.twig

@@ -163,6 +163,7 @@
             {% if versions|length %}
             {% if versions|length %}
                 <ul class="versions">
                 <ul class="versions">
                     {% for version in versions %}
                     {% for version in versions %}
+                        {% set expanded = version.id == expandedVersion.id %}
                         <li class="version{% if loop.last %} last{% endif %}" id="{{ version.version }}">
                         <li class="version{% if loop.last %} last{% endif %}" id="{{ version.version }}">
                             <section>
                             <section>
                                 <h1>
                                 <h1>
@@ -186,12 +187,12 @@
                                     <span class="license{% if not version.license %} unknown{% endif %}">{{ version.license ? version.license|join(', ') : 'Unknown License' }}</span>
                                     <span class="license{% if not version.license %} unknown{% endif %}">{{ version.license ? version.license|join(', ') : 'Unknown License' }}</span>
                                 </h1>
                                 </h1>
 
 
-                                <div class="details{% if loop.index0 == 0 %} open{% endif %}">
-                                    {% if loop.index0 == 0 %}
+                                <div class="details{% if expanded %} open{% endif %}">
+                                    {% if expanded %}
                                         {% include 'PackagistWebBundle:Web:versionDetails.html.twig' with {version: version} %}
                                         {% include 'PackagistWebBundle:Web:versionDetails.html.twig' with {version: version} %}
                                     {% endif %}
                                     {% endif %}
                                 </div>
                                 </div>
-                                <div class="details-toggler{% if loop.index0 == 0 %} open{% endif %}"{% if loop.index0 %} data-load-more="{{ path('view_version', {versionId: version.id, _format: 'json'}) }}"{% endif %}></div>
+                                <div class="details-toggler{% if expanded %} open{% endif %}"{% if not expanded %} data-load-more="{{ path('view_version', {versionId: version.id, _format: 'json'}) }}"{% endif %}></div>
                             </section>
                             </section>
                         </li>
                         </li>
                     {% endfor %}
                     {% endfor %}