Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Document title not updated when first selecting buffer #241

Open
digitalcircuit opened this issue Feb 20, 2017 · 4 comments
Open

Document title not updated when first selecting buffer #241

digitalcircuit opened this issue Feb 20, 2017 · 4 comments
Labels

Comments

@digitalcircuit
Copy link
Contributor

Issue

The call to set document.title here doesn't seem to apply when first connecting to Quassel Web and clicking a buffer. The title updates properly once a buffer is selected.

Steps

  1. Connect to Quassel Web (reload page if needed)
  2. Observe document title
  3. Select a buffer

Expected

  • Document title updates

Actual

  • Document title stays as 'Quassel Web Server'
  • The title gets correctly updated when choosing another buffer after first choosing one

I'll fill in more details later on if needed; not as much time right this moment

@magne4000 magne4000 added the bug label Feb 21, 2017
@magne4000
Copy link
Owner

Hum, I never had this behavior.

@digitalcircuit
Copy link
Contributor Author

I can still reproduce this in Firefox 53 x64 on Ubuntu Linux 16.04 LTS, no modifications (tested using the Guest login). I have multiple networks configured.

Anything that'd be helpful to collect/test?

@ohnx
Copy link
Contributor

ohnx commented Jun 4, 2017

I just tried this with quassel and Firefox 53.0.3 64bit running on Ubuntu 16.10, and didn't have any issue.

Perhaps a log of the console would help? (right click, inspect element, console)

Maybe an error is being thrown, causing the first update to not happen.

@digitalcircuit
Copy link
Contributor Author

@ohnx Thank you for the suggestion; that pointed the issue!

16:40:56.062 Error: olfbuf is null
$scope.showBuffer@https://[domain_name]/chat/javascripts/angular-controller.js:346:17
fn@https://[domain_name]/chat/javascripts/angular.min.js line 237 > Function:4:223
e@https://[domain_name]/chat/javascripts/angular.min.js:282:195
$eval@https://[domain_name]/chat/javascripts/angular.min.js:147:309
$apply@https://[domain_name]/chat/javascripts/angular.min.js:148:12
compile/</<@https://[domain_name]/chat/javascripts/angular.min.js:282:245
dispatch@https://[domain_name]/chat/javascripts/jquery.min.js:3:10263
add/q.handle@https://[domain_name]/chat/javascripts/jquery.min.js:3:8325
 1 angular.min.js:122:8
	f/< https://[domain_name]/chat/javascripts/angular.min.js:122:8
	uf/this.$get</< https://[domain_name]/chat/javascripts/angular.min.js:94:42
	$apply https://[domain_name]/chat/javascripts/angular.min.js:148:59
	compile/</< https://[domain_name]/chat/javascripts/angular.min.js:282:245
	dispatch https://[domain_name]/chat/javascripts/jquery.min.js:3:10263
	add/q.handle https://[domain_name]/chat/javascripts/jquery.min.js:3:8325

Turns out enabling the Trim buffer when switching feature causes this to break; there's no check for if olfbuf is null.

When I disable Trim buffer when switching, first selection properly updates the window title.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants