| Summary: | Vertical scrollbar always shown | ||
|---|---|---|---|
| Product: | [Frameworks and Libraries] telepathy | Reporter: | Antonio Rojas <arojas> |
| Component: | text-ui | Assignee: | Telepathy Bugs <kde-telepathy-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | kde |
| Priority: | NOR | ||
| Version: | 0.6.0 | ||
| Target Milestone: | Future | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | http://commits.kde.org/telepathy-text-ui/9f70a5428dbb66aff1770c0c4a95de31cacf6c0e | Version Fixed In: | 0.7.0 |
| Attachments: | Updated patch (replaced tabs with spaces, capitalized file name) | ||
|
Description
Antonio Rojas
2013-04-03 14:39:20 UTC
We have had a pat ch in reviewboard for absolutely ages[1], I needed someone to test it with multiple different themes. No-one has yet. If someone could do so, we can ship it. [1] https://git.reviewboard.kde.org/r/102005/ The patch fails to apply to 0.6.0 (could be because of tabs vs spaces?) After replacing tabs with spaces the patch applies correctly, and it fixes the issue for me (tested with a both themes and a few variants). The scrollbar flashes on and off when reducing the window, but that's a different issue I guess (and less noticeable) Could you upload your patch then please. Created attachment 78611 [details]
Updated patch (replaced tabs with spaces, capitalized file name)
Thanks, I'd like to see this tested on more than the 2 default themes. I'll apply this as-is now into master, so we have from now until 0.7.0 for any bug reports. If it causes problems I can remove it. Git commit 9f70a5428dbb66aff1770c0c4a95de31cacf6c0e by David Edmundson, on behalf of Paolo Capriotti. Committed on 03/04/2013 at 17:34. Pushed by davidedmundson into branch 'master'. Fixed bug where the chat window was slightly scrollable even when it fitted in the window. REVIEW: 102005 M +4 -2 data/Template.html http://commits.kde.org/telepathy-text-ui/9f70a5428dbb66aff1770c0c4a95de31cacf6c0e |