Created attachment 66725 [details]
Screenshot of Renkoo preview.
Version: git-latest (using Devel)
When using the Renkoo theme for Text-UI, the conversion can't be scrolled to the top.
The first 2 lines are always covered by the header element of the theme.
Steps to Reproduce:
1. Configure Text-UI to use the Renkoo theme
2. Make a long conversation
3. Scroll to the top
The first 2 lines aren't visible, as they're hidden behind the header element of the theme.
I'd expect to be able to see all text when scrolling to the top.
See the attached screenshot, which demonstrates the problem in the preview of the theme configuration dialog.
Seems like the theme is broken. Renkoo from http://www.adiumxtras.com/ ships with an own Template.html and doesn't have this Problem, but it adds the messages from the top and not from the bottom, like the Renkoo from text-ui does.
Created attachment 66774 [details]
Side effect of the first solution
Looking at another theme (GoneDark), there are two possible solutions if we want to keep the behavior of adding messages at the bottom of the chat-area.
Both have a side effect.
The first one adds a scroll-bar after the first message.
The second one adds a clear area with the size of the header on top if you have disabled the header.
Created attachment 66775 [details]
Side effect of the second solution
"Renkoo from http://www.adiumxtras.com/ ships with an own Template.html and doesn't have this Problem"
I believe (though could be wrong) when Renkoo became the default theme for Adium it moved into the Adium source tree, and is more up to date there. It's that version we are using in our code.
Would be interesting to see what Adium itself does.
I used to have a rule that we don't alter the themes themselves (as it implies we're doing something "wrong" in the processing) and we should be fixing that not the visible issue, however given I want to replace the whole thing eventually (this Adium theme stuff sucks heavily) I no longer mind.
What happens when you combine the second solution with https://git.reviewboard.kde.org/r/102005/diff/#index_header ?
Dominik Cermak fixed this bug with commit gc20a4db
(P.S.: If any of the admins is listening, please give karma to this good guy)