« Previous day | Index | Today | Next day » Channels | Search | Join
All times shown according to UTC.
Time | Nick | Message |
---|---|---|
03:36 | kevix has left #sugar-newbies | |
06:47 | meeting_ <meeting_!~sugaroid![]() |
|
07:04 | meeting has quit IRC | |
11:03 | meeting <meeting!~sugaroid![]() |
|
14:34 | tony37 <tony37!~tony![]() |
|
14:34 | tony37 | Hello |
14:34 | iamutkarshtiwari <iamutkarshtiwari!0e8bee62![]() |
|
14:34 | iamutkarshtiwari | #startmeeting |
14:34 | tony37 | Hello |
14:34 | meeting | Meeting started Wed Jun 22 14:34:40 2016 UTC. The chair is iamutkarshtiwari. Information about MeetBot at http://wiki.debian.org/MeetBot. |
14:34 | Useful Commands: #action #agreed #help #info #idea #link #topic #endmeeting | |
14:34 | iamutkarshtiwari | Hello |
14:34 | #JSFiddler | |
14:35 | tony37 | Did you get my email with the problems with version 15 |
14:35 | iamutkarshtiwari | Yes. |
14:35 | I'll first give you the updated of what I did the whole day | |
14:35 | updates* | |
14:35 | tony37 | OK |
14:35 | iamutkarshtiwari | I had been struggling with including all the dependencies required |
14:35 | for the html5print | |
14:36 | But I found out that there were around 28 dependencies | |
14:36 | tony37 | A lot of baggage |
14:36 | iamutkarshtiwari | which are installed during 'pip install html5print' |
14:36 | So I manually tried including all of them. | |
14:36 | Which I did successfull but.. | |
14:37 | successfully* | |
14:37 | But after including the modules | |
14:37 | inside the browse module folder, it needed me to change the relative import paths' | |
14:37 | tony37 | Yep |
14:38 | iamutkarshtiwari | so I thought |
14:38 | 'this isn't programming' what I am doing. | |
14:38 | I should automate the process. | |
14:38 | tony37 | I think of it as 'coping' |
14:38 | iamutkarshtiwari | So I added a script to the 'Browse' |
14:38 | for installing all the dependencies at the start 'for only the first time' | |
14:39 | And after that I sent you the updated XO. But as I recheck | |
14:39 | 'pip' doen't come preinstalled with Sugar. :( | |
14:39 | rechecked* | |
14:39 | doesn't* | |
14:40 | So I am againt trying to work out everything using 'only beautiful soup 4' | |
14:40 | This^ is what I had been doing all day. | |
14:40 | tony37 | Consider it an important educational experience |
14:41 | iamutkarshtiwari | Yes. :) |
14:41 | But this 'dependency" issue is barring our progress. | |
14:42 | I need to fix it. | |
14:42 | If 'pip' would have been already inclluded in the Sugar, our work had been a lot easier with that automation script I wrote. | |
14:43 | Mr. Anderson ? | |
14:44 | icarito: Hello. | |
14:45 | tony37 has quit IRC | |
14:48 | tony37 <tony37!~tony![]() |
|
14:48 | tony37 | hello, I had a glitch in the internet connection |
14:49 | iamutkarshtiwari | Ah.. nevermind :) |
14:49 | Did you receive my last text ? | |
14:49 | before quitting* | |
14:49 | tony37 | The last I got was your comment that 'pip' should have been included in Sugar. |
14:50 | iamutkarshtiwari | Yes.. So that's the problem. |
14:51 | tony37 | I disagree. Pip is for installing packages. Packages that are need by Sugar should be installed. |
14:51 | needed* | |
14:53 | Hello - am I still online? | |
14:53 | iamutkarshtiwari | Should we ask user to install |
14:53 | the dependencies by himself? | |
14:54 | Or would it be better if we automate it? | |
14:54 | tony37 | No - the idea is that Sugar is prepared for use by primary school children. They have no ordinary occasion to install software. |
14:54 | What you should learn from this experience is that some problems are not to be solved. In this case there is a reasonable alternative. | |
14:55 | Use BeautifulSoup and prettify css yourself. | |
14:55 | iamutkarshtiwari | I doubt if BeautifulSoup is capable enough to prettify css. |
14:55 | tony37 | That is why I suggested how you could do it yourself. |
14:56 | iamutkarshtiwari | For prettifying css by ourself, we neeed to add more code to the 'webconsole'. |
14:56 | tony37 | Yes, I suspect less than 20 lines |
14:57 | iamutkarshtiwari | I checked for the prettifying css code in the online modules available like 'cssprint' of html5print, and it was much bigger that |
14:57 | than* 20 lines. | |
14:58 | tony37 | Look at the email I sent. If you can't implement that in 20 lines, you will need to review your understanding of Python. This is an application at which Python excels. |
14:59 | iamutkarshtiwari | Oh... I need to look more into then. What method shall we use to prettify JS code? |
14:59 | it* | |
14:59 | tony37 | Use BeautifulSoup as you did before |
15:00 | iamutkarshtiwari | I am not whether BS was successfully able to prettify JS. |
15:00 | I'll retest it. | |
15:01 | tony37 | OK, I don't remember any issues but there may have been. That would more difficult to pretty print - but worst case set a line limit and wrap at the nearest blank. |
15:02 | iamutkarshtiwari | I'll give it a try. |
15:02 | Apart from #prettyPrinting what other issues we need to resolve? | |
15:03 | Console togger. | |
15:03 | toggler* | |
15:03 | tony37 | Did you try the html I sent you? |
15:03 | iamutkarshtiwari | I went through the code you shared for toggling the console. |
15:04 | tony37 | and displayed it on a browser? |
15:04 | iamutkarshtiwari | Yes. |
15:04 | tony37 | That is what I would like to see. |
15:04 | iamutkarshtiwari | I had already tried that method for toggling |
15:04 | the console. | |
15:04 | tony37 | And .... |
15:04 | iamutkarshtiwari | but the problem is |
15:05 | that the 'webconsole's #iframe | |
15:05 | display the webpage (generated from the user's code) dynamically. | |
15:05 | so if we place a toggle button outside the #iframe. | |
15:06 | It would only work until the page is refreshed (just before pressing 'run') | |
15:06 | But after we press 'Run' | |
15:06 | tony37 | I think you don't understand how html and css work. |
15:07 | iamutkarshtiwari | I am bit weak at html/css/js :( |
15:07 | tony37 | The two divs (in or not in an iframe) are both present in the page. The display attributes just tells the browser to process a div or skip it |
15:08 | iamutkarshtiwari | I get it. |
15:08 | I am not able to explain the problem properly I guess. | |
15:08 | tony37 | So what you need to do is to look at the generated strings (I do this sort of stuff constantly). Her generated strings consist of constant |
15:09 | parts and interspersed variables. | |
15:09 | What you will need to do is modify these strings to get what the static text I sent you does. | |
15:10 | Let her code generate the content of the two divs - your toggle only selects which one is visible. | |
15:10 | iamutkarshtiwari | Ok. I'll try to fix it that way. |
15:11 | tony37 | In the end you'll find it much easier than bundling a package! |
15:11 | iamutkarshtiwari | Bundling a package? |
15:11 | tony37 | Such as html5print |
15:11 | iamutkarshtiwari | Won't we be generating PRs? |
15:11 | oh... :D | |
15:12 | I think I am comfortabe more with Python issues than html/css/jss | |
15:12 | that's why I applied for a Python project. :) | |
15:12 | tony37 | Let me know how you are getting on - I am still looking for a new version to test. I really think we are almost home. |
15:13 | When we're finished, you can use fiddler to learn more about html and css :) | |
15:13 | iamutkarshtiwari | I'll try my best to get working. |
15:13 | tony37 | Now about the projects upcoming. |
15:14 | iamutkarshtiwari | I am comfortable |
15:14 | with working on one feature at a time. | |
15:14 | tony37 | Can you tell me how the new 'standard' alert compares to the one you now have in the 'save' part of fiddler |
15:15 | iamutkarshtiwari | Working on more than one feature distributes focus and speed. |
15:15 | First I will update you with the lastest screenshot gif. | |
15:16 | tony37 | Good |
15:16 | iamutkarshtiwari | https://github.com/sugarlabs/s[…]comment-226788259 |
15:16 | This is what it looks like now^ | |
15:17 | tony37 | My internet connection has reasonable speed (100KB/s) but terrible response time (often > 1 second). |
15:17 | iamutkarshtiwari | It's fine. I will wait :) |
15:18 | Did you see the gif ? | |
15:18 | tony37 | Ouch! As I feared it is a popup in the middle of the screen (and undoubtedly modal). |
15:18 | I really much prefer the current Sugar style of a rectangular alert just below the toolbar | |
15:19 | iamutkarshtiwari | Those regular alerts are meant to only popup inside Sugar Activities? |
15:19 | For screenshot, we need an alert which can apper(pop up) anywhere | |
15:20 | throughout Sugar. | |
15:20 | appear* | |
15:21 | tony37 | There is a different alert outside of Sugar activities? |
15:22 | iamutkarshtiwari | I dont't think there is any? |
15:22 | And community is quite satisfied with this modal popup | |
15:22 | tony37 | I would have thought the alert would be a part of the sugar toolkit |
15:22 | iamutkarshtiwari | for screenshot. |
15:23 | tony37 | At least Sam is. |
15:23 | iamutkarshtiwari | :D |
15:23 | Mr. Crossland also found it nice. | |
15:23 | tony37 | Actually there are alerts in the control panel (e.g. reminding a user to restart the laptop) |
15:24 | iamutkarshtiwari | I have already generated a PR for this popup, and it's under review (improvements) |
15:25 | tony37 | You are providing the code or are you using code from Sugar? |
15:26 | iamutkarshtiwari | I am providing most of it. I am just reusing Sam's popup module and the 'image capturing' code from previous screenshot code. |
15:26 | tony37 | So you are talking about the screenshot feature PR modified to use the code supplied by Sugar (Sam). |
15:26 | iamutkarshtiwari | Yes. |
15:27 | It looks better now. | |
15:27 | after using Sam's popup module. | |
15:27 | tony37 | Can you change the text from yes and no to save and skip |
15:28 | iamutkarshtiwari | Yes. Can we use 'Discard' instead of 'Skip' ? |
15:28 | tony37 | or cancel I was really asking does the popup allow you to specify the text |
15:29 | iamutkarshtiwari | yes. As you as the popup appears, you can direcly type in the text with moving your cursor to the text area. |
15:29 | As soon as the popup appears* | |
15:29 | tony37 | I am referring to the text in the alert not in the data entry |
15:30 | iamutkarshtiwari | Do you mean the alert's heading- 'Save screenshot' ? |
15:31 | tony37 | Yes but also the options which are now 'yes' and 'no' |
15:31 | iamutkarshtiwari | We can change all of that. |
15:32 | It's inside the code. | |
15:32 | tony37 | Great! I have seen gtk popups which only allow fixed options. |
15:32 | iamutkarshtiwari | All we need to do is change the string specified for the gkt 'buttons' |
15:33 | This one modified. And thanks to Sam's module for providing so much flexibility. | |
15:33 | ! | |
15:33 | tony37 | From a ui perspective - I would like to see an entry with a return be accepted as a 'save' or 'yes'. |
15:33 | iamutkarshtiwari | is modified version* |
15:34 | Yes. I does have this feature as well. | |
15:34 | As soon as the popup appears, | |
15:34 | the focus gets grabbed to the text entry | |
15:34 | tony37 | What happens if the user clicks on yes without making an entry? |
15:34 | iamutkarshtiwari | wherein the user types the 'name' of the screenshot. |
15:35 | and as soon as he presses 'return' the entrry gets saved. | |
15:35 | . | |
15:35 | . | |
15:35 | tony37 | That handling of the focus is important for a model. |
15:35 | iamutkarshtiwari | If the user clicks 'yes' |
15:35 | with making any entry | |
15:35 | tony37 | I am asking what happens if he moves the cursor to yes and clicks with an empty entry |
15:36 | modal* | |
15:36 | iamutkarshtiwari | the default name gets saved like 'Screenshot of' + X |
15:36 | Oh. With an empty entry.. we didn't give a thought to it. | |
15:36 | It's a nice catch! | |
15:36 | tony37 | I would prefer that either nothing happens or there is a communication that says a name is needed to save. |
15:37 | iamutkarshtiwari | I agree. But by default the text entry is set with the text 'Screeshot of X'. Same to what used to happen by default previously. |
15:39 | tony37 | In this case the user can't set the title before the alert. The alert can by default be anything, but the point is to require a user-supplied title. |
15:39 | iamutkarshtiwari | I'll then fix it to - 'nothing should happen when user clicks 'yes' without text entry' |
15:40 | tony37 | maybe the entry area could turn to a red background? |
15:40 | iamutkarshtiwari | Oh... that would look better! |
15:40 | tony37 | I believe these gtk entries can have a greyed background message. |
15:41 | iamutkarshtiwari | yes. They provide markup facility. |
15:41 | So it's possible. | |
15:41 | tony37 | Which could be 'untitled' or 'screenshot' |
15:41 | and could change to red with 'please give a title to your screenshot' | |
15:42 | Generally, Sugar assumes learners are more or less responding randomly to see what happens. I think a user would get this message quickly but it will be needed. | |
15:42 | iamutkarshtiwari | If the user clicks 'yes' on red background with 'please specify a text', what should happen? |
15:43 | tony37 | It should be ignored. |
15:43 | It is modal so the learner will have to type something or click on no | |
15:44 | So probably the simplest solution is to ignore a user with no input and then let them decide to enter a text or click on no | |
15:44 | iamutkarshtiwari | No red background? |
15:45 | tony37 | Hell yes, that will be fun! |
15:45 | iamutkarshtiwari | :/ |
15:46 | tony37 | Moving to 'save as' |
15:46 | Are you comfortable that you understand where in activity.py you need to create a new object and when it is to be a clone of the original? | |
15:47 | iamutkarshtiwari | Yes. |
15:47 | I am currently working with 'sugar-tookit' | |
15:47 | after I complete that, porting 'sugar-tookit3' activity.py | |
15:47 | shouldn't be difficult. | |
15:48 | tony37 | Agreed. |
15:48 | iamutkarshtiwari | Since most of the current sugar activities use 'activity.py' from 'sugar-tookit' (the deprecated one) |
15:48 | tony37 | Are you comfortable that you see where in activity.py you need to generate the alert? |
15:48 | iamutkarshtiwari | Yes. it's inside 'can_close()' |
15:49 | It was written so the developers could overwrite it. | |
15:49 | as per usage. | |
15:49 | tony37 | Both versions need the feature |
15:49 | iamutkarshtiwari | Thanks to the former Sugar developers. |
15:49 | Yes. | |
15:49 | override* | |
15:49 | tony37 | You may need to do it before write_file gets executed since that code writes to the Journal. |
15:50 | iamutkarshtiwari | Yes. I have almost figured out the complete working of the activity.py |
15:51 | But | |
15:51 | tony37 | Very valuable in working with Sugar activities! |
15:51 | iamutkarshtiwari | we'll have to allow activities to store the data files. |
15:51 | But after the user selects 'discard' we can delete them. | |
15:52 | *this is what Mr.Bender suggested* | |
15:52 | tony37 | Ok - but remember this refers to the data file, not the metadata. That always gets saved. |
15:53 | iamutkarshtiwari | Yes. I got it as you told me previously. Data files need to deleted but the metadata (including the entries in the Journal) needs to be saved |
15:53 | for statistical purpose. | |
15:54 | tony37 | It is game time. An interesting night since this will decide the final qualifiers for the knockout round. |
15:54 | Bye | |
15:54 | tony37 has quit IRC | |
15:54 | iamutkarshtiwari | Bye. |
15:54 | #endmeeting | |
15:54 | meeting | Meeting ended Wed Jun 22 15:54:54 2016 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot. (v 0.1.4) |
15:54 | Minutes: http://meeting.sugarlabs.org/s[…]-22T14:34:40.html | |
15:54 | Log: http://meeting.sugarlabs.org/s[…]16-06-22T14:34:40 | |
16:54 | iamutkarshtiwari has quit IRC |
« Previous day | Index | Today | Next day » Channels | Search | Join