Bug 636699

Summary: Dashboard: No max limit on dashboard tab name can result in incredibly degraded performance
Product: [Other] RHQ Project Reporter: Corey Welton <cwelton>
Component: Core UIAssignee: RHQ Project Maintainer <rhq-maint>
Status: CLOSED CURRENTRELEASE QA Contact: Corey Welton <cwelton>
Severity: medium Docs Contact:
Priority: low    
Version: 4.0.0CC: jshaughn
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-05-24 01:12:00 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
Bug Depends On:    
Bug Blocks: 585306, 637173    
Attachments:
Description Flags
In this screenshot, a tab has (mostly) recovered and taken the new tab name given. However, a breadcrumb appears at the top using an old tab name none

Description Corey Welton 2010-09-22 23:52:12 UTC
Description of problem:
There appears to be no hard limit on tab name length.  Overrunning whatever the soft limit is can result in any number of Bad Things happening to the stability of the page.

Version-Release number of selected component (if applicable):


How reproducible:
Every time

Steps to Reproduce:
1.  Create a new tab and attempt to edit the tab name.  Copy and paste the string "01234567890" a large number of times into the field.  Note the messages at the bottom that provide a success message, until the system finally begins to fail to accept the entry
2.  Attempt to delete this string, backspacing, select-all/delete, etc.
3.  View results.
  
Actual results:
Any number of things...
* Tabs that overrun the length of the tab fields
* I have seen the system get bogged down trying to make all these immediate changes, to the point of being unresponsive
* I have seen my browser session get invalidated
* I get a "Dashboard > $foobar" breadcrumb that never seems to update even after the system has recovered and the tab has been renamed to something sane. (see forthcoming screenshot)

Expected results:
* Some reasonable maximum length for a tab name.
* Perhaps if a tab has a long name, it is truncated with ellipses as to not overrun the entire tab row, i.e., "This tab name is entirely too long t..."

Additional info:

Comment 1 Corey Welton 2010-09-22 23:54:37 UTC
Created attachment 449069 [details]
In this screenshot, a tab has (mostly) recovered and taken the new tab name given.  However, a breadcrumb appears at the top using an old tab name

.

Comment 2 Jay Shaughnessy 2011-02-11 23:17:16 UTC
Should be fixed, no longer saves changes on each keystroke but rather when focus changes.

Comment 3 Corey Welton 2011-03-21 20:07:14 UTC
verified.

Comment 4 Corey Welton 2011-05-24 01:12:00 UTC
Bookkeeping - closing bug - fixed in recent release.

Comment 5 Corey Welton 2011-05-24 01:12:00 UTC
Bookkeeping - closing bug - fixed in recent release.