Improve the way Magarena popup card detail
by ubeefx
Moderators: ubeefx, beholder, melvin, ShawnieBoy, Lodici, CCGHQ Admins
Improve the way Magarena popup card detail
by vitid » 23 Oct 2013, 14:15
Hi,
I recently improve the way Magarena popup card detail to be positioned on the right panel instead of popup over the pointing card since I think it is much more easier to navigate the battlefield.This apply to both card in hand,card on battlefield,card in graveyard,remove zone and also card on stack.
Also,I add text panel over the image to display useful information such as card definition,its power/toughness.If those card is permanent on the battlefield it will also display its current power/toughness and also it currently governed ability(which retrieved from enumeration in PermanentViewerInfo.abilityFlags)
It need additional tweak,just wanna know what do you think of it? and how should I contribute this code(commit via Mercurial?Who should I notify to?)
thank,
I recently improve the way Magarena popup card detail to be positioned on the right panel instead of popup over the pointing card since I think it is much more easier to navigate the battlefield.This apply to both card in hand,card on battlefield,card in graveyard,remove zone and also card on stack.
Also,I add text panel over the image to display useful information such as card definition,its power/toughness.If those card is permanent on the battlefield it will also display its current power/toughness and also it currently governed ability(which retrieved from enumeration in PermanentViewerInfo.abilityFlags)
It need additional tweak,just wanna know what do you think of it? and how should I contribute this code(commit via Mercurial?Who should I notify to?)
thank,
- vitid
- Posts: 5
- Joined: 29 Sep 2013, 05:34
- Has thanked: 0 time
- Been thanked: 0 time
Re: Improve the way Magarena popup card detail
by Huggybaby » 24 Oct 2013, 12:30
Contact melvin, he'll probably see your post anyway.
-
Huggybaby - Administrator
- Posts: 3207
- Joined: 15 Jan 2006, 19:44
- Location: Finally out of Atlanta
- Has thanked: 701 times
- Been thanked: 594 times
Re: Improve the way Magarena popup card detail
by melvin » 25 Oct 2013, 02:11
@Huggybaby thanks for bringing this to my attention
Hi vitid, thanks for your contribution. The easiest way to getting it merged into Magarena is to create a server side clone via https://code.google.com/p/magarena/source/createClone and push your changes there. Then notify us on the dev mailing list at magarena@googlegroups.com
Do make the change configurable so we can easily switch between the current method and the new one.
The text panel needs some more work to integrate it into the UI, perhaps it should use some elements of the current theme instead of a pure white background?
Personally I prefer the popup over the card so my eyes don't have to travel all the way to the right, but that could be because I'm used to the way it works now. Another method which was recently contributed is to remove the auto popup and use the mouse wheel to trigger the popup.
Hi vitid, thanks for your contribution. The easiest way to getting it merged into Magarena is to create a server side clone via https://code.google.com/p/magarena/source/createClone and push your changes there. Then notify us on the dev mailing list at magarena@googlegroups.com
Do make the change configurable so we can easily switch between the current method and the new one.
The text panel needs some more work to integrate it into the UI, perhaps it should use some elements of the current theme instead of a pure white background?
Personally I prefer the popup over the card so my eyes don't have to travel all the way to the right, but that could be because I'm used to the way it works now. Another method which was recently contributed is to remove the auto popup and use the mouse wheel to trigger the popup.
-
melvin - AI Programmer
- Posts: 1062
- Joined: 21 Mar 2010, 12:26
- Location: Singapore
- Has thanked: 36 times
- Been thanked: 459 times
Re: Improve the way Magarena popup card detail
by Lodici » 25 Oct 2013, 12:26
My two cents.
From your screen shot you appear to have an enormous wide screen but Magarena's minimum resolution is 1024 x 600 (W x H) at which I think a permanent popup would encroach upon a busy battlefield. You also need to bear in mind the "Show cards at original size" option. Obviously if you make your feature optional then it becomes less of an issue.
The majority of the information displayed is just a duplicate of what is already displayed on the card so what is the point? However, the info displayed below the line in the second image is very useful since this shows the net P/T and any affects applied to the card and which might not be obvious from the (tiny) icons overlayed on the card image in the battlefield. Perhaps add an option to show this as a tooltip over the card on the battlefield?
- Lodici
From your screen shot you appear to have an enormous wide screen but Magarena's minimum resolution is 1024 x 600 (W x H) at which I think a permanent popup would encroach upon a busy battlefield. You also need to bear in mind the "Show cards at original size" option. Obviously if you make your feature optional then it becomes less of an issue.
The majority of the information displayed is just a duplicate of what is already displayed on the card so what is the point? However, the info displayed below the line in the second image is very useful since this shows the net P/T and any affects applied to the card and which might not be obvious from the (tiny) icons overlayed on the card image in the battlefield. Perhaps add an option to show this as a tooltip over the card on the battlefield?
- Lodici
Re: Improve the way Magarena popup card detail
by vitid » 26 Oct 2013, 03:40
Yep,I will add option in Preference menu to let you choose between each popup methodLodici wrote:My two cents.
From your screen shot you appear to have an enormous wide screen but Magarena's minimum resolution is 1024 x 600 (W x H) at which I think a permanent popup would encroach upon a busy battlefield. You also need to bear in mind the "Show cards at original size" option. Obviously if you make your feature optional then it becomes less of an issue.
The majority of the information displayed is just a duplicate of what is already displayed on the card so what is the point? However, the info displayed below the line in the second image is very useful since this shows the net P/T and any affects applied to the card and which might not be obvious from the (tiny) icons overlayed on the card image in the battlefield. Perhaps add an option to show this as a tooltip over the card on the battlefield?
- Lodici
about card size scaling I can't find easy way to do it yet, I try reuse some existing code but I get some wired error(don't know why?)
My screen is 20 inch, 1600 × 900.Screen higher that that should work fine.
- vitid
- Posts: 5
- Joined: 29 Sep 2013, 05:34
- Has thanked: 0 time
- Been thanked: 0 time
5 posts
• Page 1 of 1
Who is online
Users browsing this forum: No registered users and 39 guests