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

Fixed ticket 4686 [Version 2]. #259

Closed

Conversation

prasoon2211
Copy link
Contributor

EDIT: See #313 for the new version of the patch.

See http://bugs.sugarlabs.org/ticket/4686 for more.

Also, please see #244. This patch has the suggested improvements.

Large activity detail

sugar_large

Small activity detail

sugar_small

i5o and others added 12 commits February 15, 2014 19:22
wiki.laptop.org, where the pages used to update the activities are
located, usually serve old cached pages. We can avoid that
setting the header Cache-Control' = 'max-age=600',
and be sure the page is not older than 10 minutes.

A reason to set this at client side, and not solve it only
in wiki.laptop.org, is the posibility of intermnediate caches,
as provider or school server proxies, or the use of other
activities servers in particular deployments.

Signed-off-by: Gonzalo Odiard <godiard@sugarlabs.org>
Is better than the current font.  By the way the html version looks
better formated. We should show this in a webview:
http://www.gnu.org/licenses/gpl-2.0-standalone.html
The object_id can be retrieved in more than one chunk,
then the parser should take carr of that.

Signed-off-by: Gonzalo Odiard <godiard@sugarlabs.org>
Without this patch, the updater show a message
'Your software is up-to-date', when can't connect to the server.

Signed-off-by: Gonzalo Odiard <godiard@sugarlabs.org>
Updates LGPL-python.txt to read the FSF's new address.
This is needed to allow the next update request to work.

This was missed in the last patch "Show a error message
if the activity updater can't connect to the server"

Signed-off-by: Gonzalo Odiard <godiard@sugarlabs.org>
If the user changes the username on My Setting -> About me view and then
reverts it to the initial username, remove the warning about restart.
@prasoon2211
Copy link
Contributor Author

@manuq : Review please.

@samdroid-apps
Copy link
Contributor

Looks great, I will test.

@prasoon2211
Copy link
Contributor Author

@walterbender : I've made the changes you suggested and ran pep8 on the code.

@prasoon2211
Copy link
Contributor Author

Maybe define a constant for NUMBER_OF_LINES ?

Sure.

@prasoon2211
Copy link
Contributor Author

@walterbender : I think that was all. Anything else?

@prasoon2211
Copy link
Contributor Author

@walterbender : Done. Also, I've squashed all the previous commits into a single commit so that history doesn't clutter up. Let me know if anything more is needed.

@samdroid-apps
Copy link
Contributor

Unrelated, but how do you squash commits and keep them on the same branch?
What is your magic?
On Mar 2, 2014 4:17 PM, "Prasoon Shukla" notifications@github.com wrote:

@walterbender https://github.com/walterbender : Done. Also, I've
squashed all the previous commits into a single commit so that history
doesn't clutter up. Let me know if anything more is needed.

Reply to this email directly or view it on GitHubhttps://github.com//pull/259#issuecomment-36446500
.

@prasoon2211
Copy link
Contributor Author

@samdroid-apps : git push -f <remotename> <branchname>

@manuq
Copy link
Contributor

manuq commented Mar 5, 2014

Interesting solution, using python textwrap library.

I just hope line wrapping could be done with GTK. I think is much better that way.

I'll give a look at the gtk way. Otherwise let's just go with this.

@prasoon2211
Copy link
Contributor Author

Are you referring to the length + 1 thing (because that's the only difference I see)? Anyway, I'm not using Gtk.Label. So, we can't use set_lines

@i5o
Copy link
Contributor

i5o commented Mar 6, 2014

Wooow! Great patch!

#202 this was my try for GCI :)

2014-03-06 16:06 GMT-02:00 Prasoon Shukla notifications@github.com:

Are you referring to the length + 1 thing (because that's the only
difference I see)? Anyway, I'm not using Gtk.Label. So, we can't use
set_lines


Reply to this email directly or view it on GitHubhttps://github.com//pull/259#issuecomment-36917275
.

Ignacio Rodríguez

@dnarvaez
Copy link
Contributor

dnarvaez commented Mar 8, 2014

This really needs to be done with gtk. TextView has wrapping support too if that's what you are using.

@dnarvaez dnarvaez closed this Mar 8, 2014
@prasoon2211
Copy link
Contributor Author

@dnarvaez : Well, I had tried wrapping text using gtk but IIRC, I just couldn't get the output to be consistent. I will try some other approach with gtk, I suppose.

@dnarvaez
Copy link
Contributor

dnarvaez commented Mar 8, 2014

@prasoon2211 If you need help figuring out what is wrong feel free to post code/screenshots on the mailing list.

prasoon2211 referenced this pull request in prasoon2211/sugar Mar 27, 2014
* Used Gtk.Label with Pango based ellipsize for wrapping and
  ellipsizing.
* Fallback for Gtk < 3.10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants