maemo.org Bugzilla – Bug 11976
viewing sms conversations in portrait mode without horizontal scrolling (patch)
Last modified: 2012-12-08 14:37:13 UTC
You need to log in before you can comment on or make changes to this bug.
http://talk.maemo.org/showpost.php?p=953354&postcount=947 http://talk.maemo.org/showpost.php?p=953369&postcount=950
Thanks for doing these mods, however, including these modified CSS files might be illegal. The only way they can be accepted is a .diff file between the original file and the modified file that's applied in postinst and removed in prerm.
Sorry, I missed the second post, I'll have a look at it.
Suggested technical approach: * Package named "<name of package containing CSS files originally>-portrait" * Package includes the patch and depends on patch tools. * In postinst: ** uses dpkg-divert to move the original to "<blah>.orig" ** uses patch to merge the package patch and "<blah>.orig" to give "<blah>" * In prerm: ** Removes the diversion (putting the original file back in its original place)
The patches I attached in the post 950 are not clean. They contain a lot of commented code which have to be removed. I do not have the original files which were on my device and had to download them from another website, so I am not attaching the final patches.
(In reply to comment #4) > The patches I attached in the post 950 are not clean. They contain a lot of > commented code which have to be removed. I do not have the original files which > were on my device and had to download them from another website, so I am not > attaching the final patches. If you can attach the ideal versions to this issue, someone with an unmodified device (i.e. me) can generate clean patches to attach to the issue and someone else (anyone) can develop the "patching"/"portrait support" package.
Created an attachment (id=3302) [details] Modified CSS for SMS converstation
Created an attachment (id=3303) [details] Modified CSS for Chat converstation
Created an attachment (id=3311) [details] Patch against /usr/share/rtcom-messaging-ui/html Patch against /usr/share/rtcom-messaging-ui/html which can be applied with: cd /usr/share/rtcom-messaging-ui/html patch -p1 <portrait.patch All that needs doing is the packaging as outlined above.
With this update is there any prospect of showing a "bubble" conversation option as well similar to that shown in this post? http://talk.maemo.org/showpost.php?p=955088&postcount=791
*** This bug has been confirmed by popular vote. ***
AFAICT, the "patch" utility is only available in the SDK repository (deb http://repository.maemo.org/ fremantle/sdk free non-free). One option would be to copy it into the CSSU repository, another would be to use something like http://code.google.com/p/python-patch/ but that adds a dependency on Python.
(In reply to comment #11) > AFAICT, the "patch" utility is only available in the SDK repository (deb > http://repository.maemo.org/ fremantle/sdk free non-free). One option would be > to copy it into the CSSU repository, another would be to use something like > http://code.google.com/p/python-patch/ but that adds a dependency on Python. The "right" way is to put a copy of `patch' in either Extras or the CSSU repository (in a non-user/ section). Shouldn't be too hard, since it should only require the existing source tarball to be uploaded to the CSSU autobuilder(?)
Created an attachment (id=3346) [details] "Appendable" files, semantically equivalent to the patches (In reply to comment #12) > The "right" way is to put a copy of `patch' in either Extras or the CSSU > repository (in a non-user/ section). Shouldn't be too hard, since it should > only require the existing source tarball to be uploaded to the CSSU > autobuilder(?) Attached you will find the changes rewritten into CSS so that the data only needs to be appended to the files, no need for the patch utility. Another idea would be to place these changes somewhere in separate files and only append CSS @import statements to the files that we want to modify.
(In reply to comment #13) > Attached you will find the changes rewritten into CSS so that the data only > needs to be appended to the files, no need for the patch utility. Another idea > would be to place these changes somewhere in separate files and only append CSS > @import statements to the files that we want to modify. As discussed with jaffa on IRC (#maemo-ssu), here's the current proposal on how to deploy this: Package name: rtcom-messaging-ui-portrait Installs customized parts of .css files into: /usr/share/rtcom-messaging-ui-portrait/html/ The files have the same name as their equivalents that need to be modified (e.g. MessagingWidgetsSMSConversation.css), but only contain the additions made. The postinst script will basically need to do the following (pseudocode): for each $css_file in /usr/share/rtcom-messaging-ui/html/: $x = "@include url(../../rtcom-messaging-ui-portrait/html/$css_file);" if $css_file does not contain $x: append $x to the end of $css_file The postrm script does the same, but it removes the line from the css file if the css file contains it. The checking and removing can be done using "grep" and "grep -v", the adding can be done using echo and redirection (>>). We probably also need to use dpkg-divert, so that upgrades to rtcom-messaging-ui don't remove that part (although we could just skip that step and assume that rtcom-messaging-ui won't be upgraded any time soon, and/or release a new version of the portrait mode package if it really happens ;).
(In reply to comment #11) > AFAICT, the "patch" utility is only available in the SDK repository (deb > http://repository.maemo.org/ fremantle/sdk free non-free). One option would be > to copy it into the CSSU repository, another would be to use something like > http://code.google.com/p/python-patch/ but that adds a dependency on Python. What about adding build dependency on original package which has all CSS files and patch these files in build process (copy from /usr/... to build dir and apply patch on them)? In dsc package will be only patch (so no problem) and in builded deb package will be patched CSS files, which can be replaced by dpkg-divert in postinst script.
Git repository with patched CSS files are here: https://gitorious.org/community-ssu/rtcom-messaging-ui-portrait