You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficcontrol.apache.org by GitBox <gi...@apache.org> on 2020/10/28 15:28:24 UTC

[GitHub] [trafficcontrol] traeak opened a new issue #5211: Server page: restore to single click navigate and add "copy cell contents" menu item.

traeak opened a new issue #5211:
URL: https://github.com/apache/trafficcontrol/issues/5211


   <!--
   ************ STOP!! ************
   If this issue identifies a security vulnerability, DO NOT submit it! Instead, contact
   the Apache Traffic Control Security Team at security@trafficcontrol.apache.org and follow the
   guidelines at https://www.apache.org/security/ regarding vulnerability disclosure.
   
   - For *SUPPORT QUESTIONS*, use the Traffic Control slack (https://s.apache.org/atc-slack)
   or Traffic Control mailing lists (https://trafficcontrol.apache.org/mailing_lists).
   - Before submitting, please **SEARCH GITHUB** for a similar issue or PR.
   -->
   
   ## I'm submitting a ...
   <!-- delete all those that don't apply -->
   <!--- security vulnerability (STOP!! - see above)-->
   -  improvement request (usability, performance, tech debt, etc.)
   
   ## Traffic Control components affected ...
   <!-- delete all those that don't apply -->
   -  Traffic Portal
   
   
   ## Current behavior:
   <!-- Describe how the current features are insufficient. -->
   New functionality on the servers page requires a double click to navigate to the specific server instead of a single click that was there before.  First off there's no indication that double click is needed while single click still works on other typical pages.
   
   ## New behavior:
   <!-- Describe how the feature would improve Traffic Control -->
   In order to restore single click it might work to add a menu item for "copy cell contents" for copy/paste ability.
   
   ## Minimal reproduction of the problem with instructions:
   <!--
   If you can illustrate your feature request better with an example, please
   provide the *STEPS TO REPRODUCE* and include the applicable TC version.
   If not, feel free to delete this section.
   -->
   
   ## Anything else:
   <!--
   e.g. additional context, related issues, suggestions about how to approach (feel
   free to delete this section)
   -->
   
   <!--
       Licensed to the Apache Software Foundation (ASF) under one
       or more contributor license agreements.  See the NOTICE file
       distributed with this work for additional information
       regarding copyright ownership.  The ASF licenses this file
       to you under the Apache License, Version 2.0 (the
       "License"); you may not use this file except in compliance
       with the License.  You may obtain a copy of the License at
   
       https://apache.org/licenses/LICENSE-2.0
   
       Unless required by applicable law or agreed to in writing,
       software distributed under the License is distributed on an
       "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
       KIND, either express or implied.  See the License for the
       specific language governing permissions and limitations
       under the License.
   -->
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] ocket8888 closed issue #5211: Server page: restore to single click navigate and add "copy cell contents" menu item.

Posted by GitBox <gi...@apache.org>.
ocket8888 closed issue #5211:
URL: https://github.com/apache/trafficcontrol/issues/5211


   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] shamrickus commented on issue #5211: Server page: restore to single click navigate and add "copy cell contents" menu item.

Posted by GitBox <gi...@apache.org>.
shamrickus commented on issue #5211:
URL: https://github.com/apache/trafficcontrol/issues/5211#issuecomment-718036295


   We could detect if the mouse is held down, if so then don't open the server (to allow copying); but, if the mouse is just clicked then it will open the server as usual.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] mitchell852 commented on issue #5211: Server page: restore to single click navigate and add "copy cell contents" menu item.

Posted by GitBox <gi...@apache.org>.
mitchell852 commented on issue #5211:
URL: https://github.com/apache/trafficcontrol/issues/5211#issuecomment-718044591


   > So, I frequently want to select and copy text from the Servers page, e.g. hosts and IPs to put in a terminal or URL bar. I assume that's the problem double-click was trying to solve.
   
   yes, that's what double-click was trying to solve and yes, double click not intuitive. will revert and look at better solutions to the copy/paste pain.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #5211: Server page: restore to single click navigate and add "copy cell contents" menu item.

Posted by GitBox <gi...@apache.org>.
mitchell852 edited a comment on issue #5211:
URL: https://github.com/apache/trafficcontrol/issues/5211#issuecomment-718044591


   > So, I frequently want to select and copy text from the Servers page, e.g. hosts and IPs to put in a terminal or URL bar. I assume that's the problem double-click was trying to solve.
   
   yes, that's what double-click was trying to solve and yes, double click not intuitive. will revert and look at better solutions to the copy/paste pain which i think is a pretty common use case (the ability to c/p cell contents).


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] rawlinp commented on issue #5211: Server page: restore to single click navigate and add "copy cell contents" menu item.

Posted by GitBox <gi...@apache.org>.
rawlinp commented on issue #5211:
URL: https://github.com/apache/trafficcontrol/issues/5211#issuecomment-718030448


   +1


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficcontrol] rob05c commented on issue #5211: Server page: restore to single click navigate and add "copy cell contents" menu item.

Posted by GitBox <gi...@apache.org>.
rob05c commented on issue #5211:
URL: https://github.com/apache/trafficcontrol/issues/5211#issuecomment-718041086


   +1 - double-click is very unintuitive to me. I would've never guessed that.
   
   So, I frequently want to select and copy text from the Servers page, e.g. hosts and IPs to put in a terminal or URL bar. I assume that's the problem double-click was trying to solve.
   
   As Steve said, detect hold-downs. Could we make the UI event onmouseup instead of down, and detect if text was selected, and if so not trigger the click/navigate? That seems ideal to me, if it's possible?
   
   If not, we could change the row to not click anything, and add a column of buttons on the right or left, to click to navigate to each server's page.
   
   I'd +1 either of those, over either double-click or the current situation of trying to copy text being extremely painful.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org