Re: [Okular-devel] Review Request 123680: Fix embed mode detection

2015-05-13 Thread Albert Astals Cid

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123680/#review80320
---

Ship it!


I'm going to commit this but maybe you can also do a test case so we don't make 
the same mistake in the future?

- Albert Astals Cid


On mai. 8, 2015, 3:06 a.m., Jonathan Doman wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/123680/
 ---
 
 (Updated mai. 8, 2015, 3:06 a.m.)
 
 
 Review request for Okular.
 
 
 Repository: okular
 
 
 Description
 ---
 
 Wow, I have no idea how this happened. I wrote an big test case and spent a 
 lot of time playing with tabs. Stephan Binner even told me about this a month 
 ago, but I didn't quite understand the problem...
 
 Broken embed mode detection means that the tabbed interface is disabled. It 
 actually made it into release... ugh
 
 
 Diffs
 -
 
   core/script/kjs_document.cpp 4422953b0e157a9b0554a949969aaf9b13f44974 
   part.cpp 5b03e56e194bf84a9f02ceaf94cad13526715157 
 
 Diff: https://git.reviewboard.kde.org/r/123680/diff/
 
 
 Testing
 ---
 
 Tabs work again.
 
 
 Thanks,
 
 Jonathan Doman
 


___
Okular-devel mailing list
Okular-devel@kde.org
https://mail.kde.org/mailman/listinfo/okular-devel


Re: [Okular-devel] Review Request 123680: Fix embed mode detection

2015-05-13 Thread Jonathan Doman

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123680/
---

(Updated May 13, 2015, 10:59 p.m.)


Status
--

This change has been marked as submitted.


Review request for Okular.


Changes
---

Submitted with commit 223092aa0e1fd5fd1b48a34702d2102fdb1acccf by Albert Astals 
Cid on behalf of Jonathan Doman to branch Applications/15.04.


Repository: okular


Description
---

Wow, I have no idea how this happened. I wrote an big test case and spent a lot 
of time playing with tabs. Stephan Binner even told me about this a month ago, 
but I didn't quite understand the problem...

Broken embed mode detection means that the tabbed interface is disabled. It 
actually made it into release... ugh


Diffs
-

  core/script/kjs_document.cpp 4422953b0e157a9b0554a949969aaf9b13f44974 
  part.cpp 5b03e56e194bf84a9f02ceaf94cad13526715157 

Diff: https://git.reviewboard.kde.org/r/123680/diff/


Testing
---

Tabs work again.


Thanks,

Jonathan Doman

___
Okular-devel mailing list
Okular-devel@kde.org
https://mail.kde.org/mailman/listinfo/okular-devel


[Okular-devel] Review Request 123680: Fix embed mode detection

2015-05-07 Thread Jonathan Doman

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123680/
---

Review request for Okular.


Repository: okular


Description
---

Wow, I have no idea how this happened. I wrote an big test case and spent a lot 
of time playing with tabs. Stephan Binner even told me about this a month ago, 
but I didn't quite understand the problem...

Broken embed mode detection means that the tabbed interface is disabled. It 
actually made it into release... ugh


Diffs
-

  core/script/kjs_document.cpp 4422953b0e157a9b0554a949969aaf9b13f44974 
  part.cpp 5b03e56e194bf84a9f02ceaf94cad13526715157 

Diff: https://git.reviewboard.kde.org/r/123680/diff/


Testing
---

Tabs work again.


Thanks,

Jonathan Doman

___
Okular-devel mailing list
Okular-devel@kde.org
https://mail.kde.org/mailman/listinfo/okular-devel