fluid-work IRC Logs-2011-08-30

Owned by Fluid IRC Log Bot
Last updated: Aug 30, 2011Version comment
5 min read

[08:38:38 CDT(-0500)] <heidi_> anastasiac how would i put an icon image into my sliding panel button string? i want to have an up arrow and a down arrow for close/open. right now my <img> html is changing to a string, which might be its limitation?
[08:41:37 CDT(-0500)] <heidi_> here's the mockup of what i'm trying to do: http://wiki.fluidproject.org/download/attachments/26969627/IDI+site+homepage-01.png ... so that gear icon i can make a bg image on the button, but not sure about the arrows
[08:51:14 CDT(-0500)] <heidi_> here's what i have so far: http://dev.inclusivedesign.ca/wordpress/
[08:58:24 CDT(-0500)] <anastasiac> heidi_, I'm just back now, looking at your question, and my initial answer is: Good question, I don't know! we'll have to figure that out (smile)
[08:58:49 CDT(-0500)] <anastasiac> heidi_, you can probably use background images in the css. Have you tried that?
[08:59:10 CDT(-0500)] <heidi_> anastasiac how would i differentiate between and up and down status? and also i already have a bg image on the button
[08:59:12 CDT(-0500)] <anastasiac> ah, right, you want the arrows to change
[08:59:15 CDT(-0500)] <anastasiac> hm...
[08:59:54 CDT(-0500)] <anastasiac> well, the "fluid" way of doing that would be for the component to adjust a style class for you. we'll have to check if that happens. if not, I think it should
[09:00:31 CDT(-0500)] <heidi_> right, like to add fl-toggleButton-open and fl-toggleButton-closed to the button
[09:00:56 CDT(-0500)] <heidi_> i wonder if i could have two bg images tho...hm
[09:01:29 CDT(-0500)] <heidi_> i don't think so, i think it would over-ride the gear
[09:01:35 CDT(-0500)] <heidi_> unless we put a span around the text
[09:01:39 CDT(-0500)] <heidi_> inside the button
[09:01:42 CDT(-0500)] <heidi_> with those classnames
[09:02:44 CDT(-0500)] <anastasiac> heidi_, it looks like the code does NOT programmatically adjust a style class on the button, but to me, it should. If it did do that, do you think you'd be able to manage the css for it?
[09:02:59 CDT(-0500)] <anastasiac> maybe you'd need one backbround image that contained the gears and the arrow?
[09:03:30 CDT(-0500)] <heidi_> anastasiac don't think one image because the width between the icons will change depending on text size
[09:03:36 CDT(-0500)] <anastasiac> ah, right
[09:03:39 CDT(-0500)] <heidi_> but a span around the string might do it
[09:03:58 CDT(-0500)] <heidi_> <span class="fl-toggleButton-open">Display Settings</span>
[09:04:16 CDT(-0500)] <anastasiac> we should probably talk to Justin_o about the lack of style class. not sure how important it is for this release
[09:04:21 CDT(-0500)] <heidi_> or maybe a div cos i don't think a span could have padding
[09:04:51 CDT(-0500)] <heidi_> i guess it is an extra feature
[09:05:32 CDT(-0500)] <heidi_> i can change the text to be Show Display Settings/Hide Display Settings for the time being...
[09:07:24 CDT(-0500)] <Justin_o> anastasiac, heidi_: if the style class is necessary we can add it for this release
[09:07:44 CDT(-0500)] <heidi_> Justin_o it's necessary to add an up/down arrow to the tab
[09:07:57 CDT(-0500)] <heidi_> but not sure if THAT is necessary
[09:08:00 CDT(-0500)] <heidi_> for this release
[09:08:52 CDT(-0500)] <Justin_o> heidi_: it's seems reasonable for it to be there.. i'm not opposed to adding it in.. i don't think that it will be very difficult to add either..
[09:09:01 CDT(-0500)] <heidi_> great
[09:10:20 CDT(-0500)] <heidi_> Justin_o anastasiac i'll make a jira
[09:11:01 CDT(-0500)] <Justin_o> heidi_: thanks
[09:11:16 CDT(-0500)] <heidi_> Justin_o assign to you?
[09:12:30 CDT(-0500)] <Justin_o> heidi_: probably leave it unassigned for now, but please add the "Bug Parade Infusion" comment to it and set the fix for to 1.4
[09:12:38 CDT(-0500)] <heidi_> k
[09:14:17 CDT(-0500)] <heidi_> Justin_o fyi http://issues.fluidproject.org/browse/FLUID-4410
[09:21:16 CDT(-0500)] <Justin_o> heidi_: thanks
[10:17:46 CDT(-0500)] <cindyli> Justin_o, michelled, i sent a pull request on 4408 for the fix on the builder issue. can one of you review the change? thanks.
[10:20:40 CDT(-0500)] <Justin_o> cindyli: just read over your commit log for FLUID-4408, when you build packages do the generated css files get added?
[10:20:50 CDT(-0500)] <cindyli> yes
[10:21:43 CDT(-0500)] <Justin_o> cindyli: what does the shell script do?
[10:22:11 CDT(-0500)] <cindyli> Justin_o: quite a number of things. let me bring up the script
[10:22:38 CDT(-0500)] <Justin_o> thanks
[10:24:29 CDT(-0500)] <cindyli> Justin_o: 1, export builder, 2. clone infusion, 3. run ant build of "daily" target
[10:24:52 CDT(-0500)] <cindyli> the last step is the one we wanna focus
[10:25:01 CDT(-0500)] <cindyli> on this issue
[10:25:20 CDT(-0500)] <Justin_o> cindyli: okay..
[10:25:38 CDT(-0500)] <Justin_o> is that target run so that the builder can work?
[10:25:58 CDT(-0500)] <cindyli> that target generates all uiOptions css files
[10:26:28 CDT(-0500)] <cindyli> in builder website, when you click download, "builderBuild" target gets run
[10:26:49 CDT(-0500)] <Justin_o> cindyli: okay.. so builderbuild actually only does a partial build.. it's expecting something to have run before.. and that thing already ran the generation target
[10:26:52 CDT(-0500)] <Justin_o> is that correct?
[10:27:00 CDT(-0500)] <cindyli> exactly
[10:28:51 CDT(-0500)] <cindyli> Moreover, as i mentioned in the commit msg, another reason that "generateUIOThemes" is excluded from "builderBuild' target is the permission problem
[10:29:12 CDT(-0500)] <cindyli> builder site uses "www-data" to create and access folders and files
[10:29:49 CDT(-0500)] <cindyli> "components/uiOptions/css" is created and owned by daily shell script with user 'continuum'
[10:30:06 CDT(-0500)] <cindyli> 'www-data' doesn't even have write permission to that directory
[10:30:56 CDT(-0500)] <cindyli> we could grant it the permission with 777 or maybe 775, but may result in security issue
[10:31:32 CDT(-0500)] <Justin_o> cindyli: okay.. i see what you're saying.. after standup can i test it out with you a bit
[10:31:38 CDT(-0500)] <cindyli> sure
[10:31:38 CDT(-0500)] <Justin_o> i think the rest seems to look reasonable
[10:31:41 CDT(-0500)] <Justin_o> thanks
[10:31:43 CDT(-0500)] <cindyli> np
[13:04:19 CDT(-0500)] <huslage> can we get this in fluid? http://www.studiostudio.nl/project-dyslexie/
[13:33:50 CDT(-0500)] <Justin_o> huslage: i was looking at that before.. pretty neat if it works as advertised and I think would be great to have for our UIO component
[13:34:26 CDT(-0500)] <huslage> i've talked to a bunch of folks (another one today) who were surprised how well it worked. They've never been able to read like that before. one friend switched it to be her default font.
[13:34:42 CDT(-0500)] <huslage> so anecdotally it works at least
[13:38:47 CDT(-0500)] <Justin_o> huslage: that's good to hear.. it's more believable when you hear it from people who actually use it
[13:38:55 CDT(-0500)] <huslage> yeah
[13:39:06 CDT(-0500)] <huslage> i couldn't tell if it worked, so it was good
[13:39:12 CDT(-0500)] <huslage> it just looks like a font to me
[13:39:41 CDT(-0500)] <Justin_o> yah.. me too.. they explained it well in the video but i was still a bit skeptical
[14:59:48 CDT(-0500)] <michelled> Justin_o: I just wanted to let you know that Bosmon had asked me if it was ok to push the file perms changes from yesterday: https://github.com/fluid-project/infusion/commit/e137c1fb1bd502ac92432bb5ab72e827c98265f0
[15:00:00 CDT(-0500)] <michelled> I said yes and promptly forgot
[15:00:02 CDT(-0500)] <michelled> oops
[15:00:13 CDT(-0500)] <michelled> anyway, I've reviewed the modifications and they are fine
[15:00:17 CDT(-0500)] <Bosmon> I am not a complete renegade......
[15:00:19 CDT(-0500)] <Bosmon> Only 50% of one (tongue)
[15:00:39 CDT(-0500)] <michelled> Justin_o: I guess in the future we'll go down the correct path and use a pull request
[15:00:50 CDT(-0500)] <michelled> and review before stuff gets into the repo - not after
[15:02:34 CDT(-0500)] <Justin_o> michelled, Bosmon: thanks.. it would be preferred to follow the standard procedure.. thanks for looking it over though and for the fixes