rawlinp commented on a change in pull request #6014:
URL: https://github.com/apache/trafficcontrol/pull/6014#discussion_r672407260
##########
File path: traffic_portal/app/src/common/modules/form/type/form.type.tpl.html
##########
@@ -29,6 +29,16 @@
<button class="btn btn-primary" title="View Cache Groups"
ng-show="type.useInTable == 'cachegroup'" ng-click="viewCacheGroups()">View
Cache Groups</button>
<button class="btn btn-primary" title="View Static DNS Entries"
ng-show="type.useInTable == 'staticdnsentry'"
ng-click="viewStaticDnsEntries()">View Static DNS Entries</button>
</div>
+ <div class="btn-group pull-right" role="group" uib-dropdown
is-open="more.isopen">
+ <button name="moreBtn" type="button" class="btn btn-default
dropdown-toggle pull-right" uib-dropdown-toggle aria-haspopup="true"
aria-expanded="false">
+ More
+ <span class="caret"></span>
+ </button>
+ <ul class="dropdown-menu-right dropdown-menu pull-right"
uib-dropdown-menu>
+ <li role="menuitem"><button class="menu-item-button"
type="button" ng-show="type.useInTable == 'server'"
ng-click="queueUpdatesByType()">Queue Server Updates By type</button></li>
+ <li role="menuitem"><button class="menu-item-button"
type="button" ng-show="type.useInTable == 'server'"
ng-click="clearUpdatesByType()">Clear Server Updates By Type</button></li>
+ </ul>
+ </div>
Review comment:
I noticed that the `More` button for Types is on the left while most forms
have it on the right after all the other buttons. Looking at Profiles, it seems
like it might need to be within the `div` tag that starts on L26, but I'm not
sure :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
