Coding Projects and Duga Heaven :)

Duga

12:00 AM

RELOAD!

[Cardshifter/Cardshifter] 2 commits. 5 opened issues. 6 issue comments.

[retailcoder/Rubberduck] 21 commits. 8 opened issues. 6 closed issues. 6 issue comments.

[Zomis/Duga] 1 opened issue. 1 closed issue. 3 issue comments.

Phrancis

12:12 AM

card('NUWA') {
    creature "Chinese"
    flavor "Serpent Creator Goddess of Mankind and Repairer of the Pillar of Heaven."
    health 10
    sickness 1
    manaCost 25
    attack 4
    afterPlay {
        summon 1 of "Terracotta Soldier" to "you" zone "Battlefield"
        summon 1 of "Manchu Archer" to "you" zone "Battlefield"
    }
    whilePresent {
        pickAtRandom (
            { heal 2 to "you" },
            { heal 1 to "opponent" },
            { damage 1 to "you" },
            { damage 2 to "opponent" }

Duga

12:27 AM

[Cardshifter/Cardshifter] Phrancis pushed commit dd72f4a8 to js: Early Chinese cards

[Cardshifter/Cardshifter] Phrancis pushed commit c20b0c4a to js: #231 Chinese deck

[Cardshifter/Cardshifter] build #1008 for commit dd72f4a8 on branch js passed

[Cardshifter/Cardshifter] build #1009 for commit c20b0c4a on branch js passed

Duga

1:17 AM

[Cardshifter/Cardshifter] Phrancis pushed commit 02b021d4 to js: Update chinese.groovy

[Cardshifter/Cardshifter] build #1010 for commit 02b021d4 on branch js passed

Stack Exchange

And another feature completed. #NextReleaseRocks #Navigate #VBA http://t.co/2KLQbaENeF

Tweeted by rubberduckvba on May 23, 2015 at 1:24 AM

I can't believe I can do all that in the VBE. @Rubberduck203 haven't we come a long way! #VBA #Refactor #Navigate http://t.co/3MmYJZfaRS

Tweeted by rubberduckvba on May 23, 2015 at 1:32 AM

Vogel612

10:49 AM

@Phrancis you around?

about these commands... feel free to post them to JavaBot as suggestions.

But... as of now the bot doesn't work properly for the stackexchange chat domain...

skiwi

hey

Vogel612

hiya

skiwi

@Phrancis Is it me or is every card from Mythos "special"? Not that it would be wrong, but I see lots of afterPlay/whilePresent effects

@Vogel612 Have you had time to look at it, or not so much?

Vogel612

not so much.

mostly because we plan to migrate to JChatExchange anyways and "wasting" too much time on it would be rather unrewarding

then again JChatExchange is currently somewhat stalled, because the creator doesn't want code of the current quality associated with their name...

and I don't have much time on my hands.

I have something in progress for 2 weeks now, with a real effort of 1 day, or maybe 2.

skiwi

Hmm okay

Can't help out much either here

Kind aoccupied with the university project

Vogel612

11:00 AM

I'll make myself some free time later in the summer

and I'll use it to get JChatExchange into a nice committable and working state.

which should make the workarounding in the Bot moot

skiwi

I hope to be able to make awesome websites soon (tm)

I'm already working on them... this project might go open-source, but right now it isn't

Simon André Forsberg

11:44 AM

@Phrancis How do you intend a heal or damage to work while inside whilePresent ??

skiwi

11:58 AM

@SimonAndréForsberg Maybe he means per turn... but whilePresent is very weird indeed and needs clarification ;)

Simon André Forsberg

whilePresent means something that is in action always, as long as the card is on the battlefield. Heal and damage are one-time effects, not something that can be inside whilePresent

skiwi

2:01 PM

@SimonAndréForsberg Is there some modular way you can define the heal and damage modifiers to only apply to "single-time" events, rather than whiel present events?

Simon André Forsberg

@skiwi @Phrancis code would cause an error while executed. So in a way, I already have.

bazola

if it was atTurnEnd instead of whilePresent would that work?

Simon André Forsberg

2:23 PM

yes

skiwi

@SimonAndréForsberg But that's only because those methods are not defined on the whilePresent method?

Simon André Forsberg

2:49 PM

@skiwi you can't define method on a method. Not even in groovy. Those methods are not defined in the WhilePresentDelegate. See github.com/Cardshifter/Cardshifter/blob/groovy/cardshifter-core/… and github.com/Cardshifter/Cardshifter/blob/groovy/cardshifter-core/…

skiwi

3:21 PM

@SimonAndréForsberg But you can make those map entries dynamic?

I can see situations where you both in a whilePresentDelegate and onEndOfTurnDelegate want to reuse the same code

Even though... interesting

Can't you capture what stuff is present inside a delegate, rather than mapping with a map?

Simon André Forsberg

3:37 PM

@skiwi of course it's possible to reuse some code.

@skiwi What do you mean by that?

skiwi

3:49 PM

@SimonAndréForsberg Right now you return a map that some keys and values in it

Can't you instead of returning a map with hard-coded keys, instead create a method that takes in the stuff inside your delegate as parameters?

afterPlay {
    summon 1 of "Terracotta Soldier" to "you" zone "Battlefield"
    summon 1 of "Manchu Archer" to "you" zone "Battlefield"
}

In this case on the AfterPlayDelegate you would then get a list with two entries, where one entry is a map like ["summon", ...]

Simon André Forsberg

@skiwi Where do I do that??

Oh dear, I think I see the confusion.

github.com/Cardshifter/Cardshifter/blob/groovy/cardshifter-core/… does not really do what you think it does.

skiwi

In Groovy you have map literals... and the last statement in a method gets returned, right?

Simon André Forsberg

yes

I am however not really returning a map in the traditional sense. I am returning a way for the summon 1 of 'Yadayada' to 'you' zone 'Battlefield' to work.

skiwi

But is it not also a map in traditional sense at the same time?

I understand the logic of your current methods regarding DSL

but is it not possible to take the DSL arguments as input, rather than to provide a static map?

Simon André Forsberg

4:06 PM

I am not really "returning a map that has some keys and values in it". I am making it possible to use the DSL methods. When calling the DSL methods, something is added to a list, which is then executed in the game at the right moment.

@skiwi How would you expect that to be coded?

skiwi

@SimonAndréForsberg More or less a funtion accepting the string/tokens summon 1 of "Terracotta Soldier" to "you" zone "Battlefield" as input

As we talked about having the option to reorder fragments earlier and also about using "order by" rather than "orderBy", how could that be supported at all in the current way?

Simon André Forsberg

@skiwi The way to accomplish that is to use def summon(Map arguments), in which case it could be called by this:

summon amount: 1, of: "Manchu Archer", to: "you", zone: "Battlefield"

which would not be an improvement IMO

skiwi

With that boilerplate, indeed not an improvement

(The colons and commas)

Simon André Forsberg

@skiwi It's not supported. Making it "order by" would complicate things a lot

skiwi

While thinking about it... say one wants to absolutely insist on offering that functionality, then pre-processing a file to replace "order by" with "orderBy" would be simplest?

Simon André Forsberg

4:12 PM

@skiwi simplest yes. clean? no. nice? no. worth it? no.

skiwi

When I was reading about DSL I found this however doesn't look like you are using it?

docs.groovy-lang.org/docs/latest/html/documentation/…

Simon André Forsberg

and it would cause pain with things like this: flavor: 'Something order by something equals something'

skiwi

That seems to allow fragment reuse

bazola

i think that part of the idea of a DSL is to formalize the terms/worlds that people have to use to build cards? so having multiple ways to say things seems more confusing than anything to me

Simon André Forsberg

@skiwi you mean the @DelegatesTo or the Builder pattern? I'm using @DelegatesTo somewhere (but will add more of those), and not using much builder pattern either.

skiwi

4:15 PM

The builder pattern

By the looks of it it seems more flexible than that map pattern (I don't even see it documented in this document)

Simon André Forsberg

slightly more flexible, yes. But I haven't really prioritized making builder classes. I found that the "map pattern", if you can call it that, is simple enough and quite easy to use.

the only benefit is really that you can write things in the order you want.

skiwi

Fair enough, but doesn't give IDE support then?

Simon André Forsberg

what about IDE support? What do you mean?

skiwi

Code completion, etc.

Simon André Forsberg

with modders writing stuff using the DSL?

skiwi

4:20 PM

I'm not sure when code completion is applicable

When you use DSL's in your own application as code files, then probably

No clue if we can somehow make that available to modders

Simon André Forsberg

might be partially supported later when more @DelegatesTo are added, but given the dynamicness of Groovy, full code completion is not possible.

skiwi

(Depends on if they write in IDE or plain-text)

@SimonAndréForsberg Full code completion on DSL's is possible, as I read it

Simon André Forsberg

depends on how much of Groovy's dynamic typing you use. Cardshifter uses that quite a bit.

skiwi

Such a shame we are stuck on XML/Python for our configuration (which is very complex in our project), Groovy's DSL seems to be the best choice then

Simon André Forsberg

I think it's easy enough to use plain-text, I don't see much need to support full code completion. Wasn't the whole point of starting doing the JS modding that most modders don't use an IDE?

skiwi

4:25 PM

@SimonAndréForsberg I already forgot the point

Simon André Forsberg

Apparently you did.

Phrancis

5:47 PM

@Vogel612 OK cool

skiwi

hey @Phrancis

Phrancis

@skiwi I might have went a bit overboard with the effects, but hard to say until we can actually play it

@SimonAndréForsberg Um, did I do that? If so I probably meant to use onEndOfTurn...

You have probably notice that I used the summon feature quite a bit for the Chinese (it just seemed to fit with the stories) so likely other pantheons will have a way different approach

(and in part why I'd like for others to come up with their ideas, including you @skiwi if you find the time)

skiwi

I'm not a content creator though :P I'm a programmer

Though I can have lots of inspiration at times

Duga

[libgdx/libgdx] badlogic created tag 1.6.1

Phrancis

Well, it's on the table. The Roman are unclaimed and their mythology is familiar, making the cards themselves takes very little time (mine took longer because I documented each card, I don't expect others to do that at all)

Duga

6:05 PM

[Cardshifter/Cardshifter] Phrancis commented on issue #231: "Mythos" decks

> To clarify one thing, I don't expect others to document each card the way I did for the Chinese, if you just come up with the card attributes and effects, I'll take care of the rest later. It'll be a while yet before we support images and such so it's no hurry.

@Duga has nothing to remind anyone of today!

skiwi

@Duga Why do you keep reminding us of that?!

Stack Exchange

@samharwell mine too! ...wait no, my rubberduck has github issues, not the same thing! Keep up the great work on ANTLR, #AntlrIsAwesome

Tweeted by rubberduckvba on May 23, 2015 at 6:15 PM in reply to samharwell

Duga

6:39 PM

[Cardshifter/cardshifter.github.io] Phrancis opened issue #51: Side menu does not display properly on mobile

> I pulled the site on my iPhone and the side navigation menu is displayed _on top_ of the page content. I'm not sure how to fix it exactly, I suspect it could even be caused by Bootstrap in order to make the page less wide.

A possible fix might be to extend the left nav menu bar all the way to the bottom of the page (albeit with empty space). See attached screenshot.

I would be curious if this problem also happens on Android, is anyone able to test pulling up http://stats.zomis.net/io-we

[Cardshifter/cardshifter.github.io] Phrancis added label help wanted to issue #51: Side menu does not display properly on mobile

[Cardshifter/cardshifter.github.io] Phrancis added label bug to issue #51: Side menu does not display properly on mobile

[Cardshifter/cardshifter.github.io] Phrancis added label page layout to issue #51: Side menu does not display properly on mobile

Phrancis

@Marc-Andre ^^ next time you're around, if you have any idea please let me know

Duga

[Cardshifter/cardshifter.github.io] skiwi2 commented on issue #51: Side menu does not display properly on mobile

> The same issue appears to occur on Android:

![screenshot_2015-05-23-20-43-50](https://cloud.githubusercontent.com/assets/2961606/7785357/a4c15c82-018c-11e5-8142-d5dc878eaa05.png)

[Cardshifter/cardshifter.github.io] Phrancis commented on issue #51: Side menu does not display properly on mobile

> For reference, this is what it _should_ look like:
![screen shot 2015-05-23 at 2 45 33 pm](https://cloud.githubusercontent.com/assets/8539648/7785361/6f0265fa-015a-11e5-92e1-e9a690fda898.png)

skiwi

@Phrancis Wow, you have many tabs open there ;)

Phrancis

Look who's talking ;)

Hosch250

There you go.

@Phrancis Anything wrong?

Is Cardhisfter a Windows Phone Exclusive?

3

Phrancis

6:57 PM

LOL

I think Marc-Andre fixed that, just need to update the server

Duga

[Cardshifter/cardshifter.github.io] Phrancis commented on issue #51: Side menu does not display properly on mobile

> Bug also happens on Windows phone:

![zqobg49](https://cloud.githubusercontent.com/assets/8539648/7785400/4a8867ae-015c-11e5-85cb-8efe8c7cbb8a.png)

Phrancis

Thanks @Hosch250 much appreciated

skiwi

Doesn't @Hosch250 mean that it doesn't happen on Windows Phone?

Phrancis

8 mins ago, by Hosch250

@skiwi Same problem there

skiwi

Ah nevermind

Eh WTF?

Why is it called Cardhisfter??

Phrancis

7:01 PM

3 mins ago, by Phrancis

I think Marc-Andre fixed that, just need to update the server

skiwi

Oh crap, I totally did not see that typo when Hosch just pointed that out

Phrancis

I have to say though, pretty bad typo, especially showing as top header on the landing page ;)

Hey @Marc-Andre :D

Marc-Andre

Sorry @Phrancis was away for a while our cat was sick and died yesterday! But I don't know what you expect it to look on mobile

But hey everyone! :D

Phrancis

Oh man, sorry to hear about that; losing a pet is always painful

Marc-Andre

Oh yeah I know it is! We had it for two years but he was part of the family already :(

Thanks @Phrancis

Phrancis

7:07 PM

Had to put a dog to sleep a few years ago because he was so sick

Marc-Andre

At least we did not have to take that decision. He was at the vet hospital and died there (we though he was doing better but he died in the night (that does not feel like the right way to say this :S ))

Phrancis

I'm pretty sure nesting statements in multiple sets of parentheses in plain English is a programmer thing

But I feel you

Marc-Andre

The funny thing is, his name was Linux and today I finally installed Linux on my computer :P

Phrancis

@Marc-Andre In any case, have you seen something like that with Bootstrap before? Is that actually what should be expected?

Marc-Andre

I think the way the layout works at the moment it is what it should look like.

Phrancis

7:12 PM

<aside class="col-sm-3 col-md-2 sidebar csh-side-nav" data-th-fragment="left-side-navigation">
  <nav class="nav nav-sidebar">

The csh-side-nav only has things like text font and color, so I don't think it has anything to do with it

Marc-Andre

Yeah it how we set up the layout with bootstrap

Phrancis

Is it a case of works-as-intended then?

Marc-Andre

It's a case of works-as-it-is-but-maybe-not-what-we-want

Phrancis

Ah.

Marc-Andre

I don't see how we would have a side menu and the content side by side though

Mobile are not large to fit that many content

Phrancis

7:18 PM

Might be right, seems like it probably would "squish" everything together so closely that the page would become almost unreadable?

Marc-Andre

tutorial.shiksha360.com/bootstrap-layout-grid-system

Look at this and it behave exactly like we have

Duga

[Cardshifter/cardshifter.github.io] Phrancis commented on issue #51: Side menu does not display properly on mobile

> Looks like this is not a bug and actually functions as it should according to Bootstrap grid system. For reference: http://tutorial.shiksha360.com/bootstrap-layout-grid-system/

Perhaps at some point we should actually convert this to a top menu with drop-downs instead of a side nav... thoughts?

[Cardshifter/cardshifter.github.io] Phrancis removed label bug from issue #51: Side menu does not display properly on mobile

[Cardshifter/cardshifter.github.io] marcandregirard commented on issue #51: Side menu does not display properly on mobile

> The thing here is I don't think the side menu would fit with the content. There is not enough place to show them both side by side. I think it may look like a bug, but the result is what is better for mobile. What we could have is some links on mobile to get to the top of the page (the side menu) quickly to help the user navigate.

Phrancis

7:50 PM

@Marc-Andre Looks at my other website, psychoproductions.net

It's made with Wordpress obviously, but something like that could be done with Bootstrap right? Like the navs at the top with dropdowns

Marc-Andre

I'm not sure dropdown is the easiest thing on mobile for the UX

Phrancis

I just pulled up that website on my phone, it looks really screwed up. Ugh

Maybe we should just leave "well enough" alone

Come to think of it, browsing websites on mobile really is a PITA

Perhaps more of a problem with the medium rather than the implementation

Marc-Andre

Indeed it's hard to make site mobile-friendly !

skiwi

8:09 PM

Can't the mobile side/top-menu be made collapsible?

Marc-Andre

Well the thing is at the moment some top level element on the nav are links. So we can't click to collapse/uncollapse all elements

skiwi

Ah...

Simon André Forsberg

11:33 PM

@Phrancis ah, I like the thought of different themes for the different mythologies!

Phrancis

Indeed

Simon André Forsberg

@skiwi because someone is too lazy to change the database? and to make it easier for me to add a reminder in case it's needed :) (I could add one for @Phrancis, but I forgot what he was supposed to remember...)

Phrancis

^ Post-It note did the trick ;D

Simon André Forsberg

@Marc-Andre Sorry for your loss

@Marc-Andre Now you will always remember him :)

Duga

Simon André Forsberg vs. Mat's Mug: 899 diff. Year: -42. Quarter: -792. Month: -486. Week: -210. Day: -20.

rolfl vs. 200_success: 292 diff. Year: +1012. Quarter: +855. Month: +476. Week: +40. Day: -30.

Loki Astari vs. Simon André Forsberg: 3925 diff. Year: -1851. Quarter: -552. Month: -169. Week: +255. Day: +40.

Simon André Forsberg

11:50 PM

@Duga ouch, 899? Mug is catching up on me

Transcript for

Coding Projects and Duga Heaven :)