Bug 836800 - console title too long
console title too long
Product: Fedora
Classification: Fedora
Component: guake (Show other bugs)
All All
unspecified Severity low
: ---
: ---
Assigned To: Pierre-YvesChibon
Fedora Extras Quality Assurance
Depends On:
  Show dependency treegraph
Reported: 2012-07-01 10:17 EDT by Ruggero
Modified: 2012-07-01 12:11 EDT (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2012-07-01 11:46:13 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Ruggero 2012-07-01 10:17:00 EDT
Description of problem: the title of the console is too long

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

How reproducible:

Steps to Reproduce:
1. cd directory1
2. cd directory2
3. ...

Actual results:
the title in the bar is too long, it should be better to truncate it

Expected results:

Additional info:
my $PS1 is
Comment 1 Pierre-YvesChibon 2012-07-01 11:46:13 EDT
I think it relies on your PS1, so this is up to you to customize it.

You may want to bring this to upstream but as far as Fedora is concerned, this is not a bug to me.
Comment 2 Ruggero 2012-07-01 12:00:38 EDT
yes, it is not a bug, but I think you should fix it. With some special PS1 (for example mine) guake is unusable. Also for very long directory.

I think you should add something like this:

string shorten_title(const string & console_title_display, int max_length)
   return console_title_display.substr(0, max_length-3) + "...";

string console_title = ...
string console_title_display = console_title;
if (console_title_display > MAX_TITLE_LENGTH) {
   console_title_display = shorten_title(console_title_display,
Comment 3 Pierre-YvesChibon 2012-07-01 12:11:53 EDT
Feel free to propose a patch upstream to shorten it when needed, the sources are on both github and gitorious.

Note You need to log in before you can comment on or make changes to this bug.