Double click research

Discussion in 'Feedback' started by PredatoR[HUN], Dec 12, 2011.

  1. PredatoR[HUN]

    PredatoR[HUN] Member

    Messages:
    1,704
    Likes Received:
    9
    Trophy Points:
    0
    Right now if you want to research something you need to select it and then move you mouse down to the "research" button. I say skip that button and make it so you can start a research simply by double clicking it.
     
    PredatoR[HUN], Dec 12, 2011
    #1
  2. w00kie

    w00kie Mustachioed Mexican

    Messages:
    3,863
    Likes Received:
    13
    Trophy Points:
    0
    or make atleast replace the button with a research ON <-> research OFF switch.
    [​IMG]
     
    w00kie, Dec 12, 2011
    #2
  3. flasche

    flasche Member Staff Member Moderator

    Messages:
    13,299
    Likes Received:
    168
    Trophy Points:
    0
    good one pred
     
    flasche, Dec 12, 2011
    #3
  4. McGyver

    McGyver Experimental Pedagogue

    Messages:
    6,533
    Likes Received:
    31
    Trophy Points:
    0
    Probably 30000 lines of GUI code need to be rewritten to include this suggestion.
     
    McGyver, Dec 12, 2011
    #4
  5. LordDz_2

    LordDz_2 Strange things happens here

    Messages:
    2,956
    Likes Received:
    93
    Trophy Points:
    0
    phff, just copy and paste! Moar code is always good, can't mess up anything.
     
    LordDz_2, Dec 12, 2011
    #5
  6. FN198

    FN198 Member

    Messages:
    2,434
    Likes Received:
    0
    Trophy Points:
    0
    either the scary truth or a good excuse
     
    FN198, Dec 12, 2011
    #6
  7. flasche

    flasche Member Staff Member Moderator

    Messages:
    13,299
    Likes Received:
    168
    Trophy Points:
    0
    from what i heard from former devs, sadly its the first one ...
     
    flasche, Dec 12, 2011
    #7
  8. Nickierv

    Nickierv Member

    Messages:
    510
    Likes Received:
    0
    Trophy Points:
    0
    While we are on fixing the research ui, can we please move the "cancel research" to not be next to "open research tree". No idea how this got past testing for so long, but as this will require a major rewrite of empires...
     
    Nickierv, Dec 13, 2011
    #8
  9. Beerdude26

    Beerdude26 OnThink(){ IsDownYet(); }

    Messages:
    7,243
    Likes Received:
    13
    Trophy Points:
    0
    It's more like this:

    Rewrite about 80 loc (lines of code) to even just get the image in, because the research panel is assembled with hand-made code. Seems simple, until you realize it looks like this:

    Adding a single image probably implies changing nearly every single one of those values. There are better ways to waste one's time :p

    Compare this with the layout code for the research overview you get at the beginning of each round:

     
    Last edited: Dec 18, 2011
    Beerdude26, Dec 18, 2011
    #9
  10. LordDz_2

    LordDz_2 Strange things happens here

    Messages:
    2,956
    Likes Received:
    93
    Trophy Points:
    0
    That makes me want to throw up. And I'm not good at coding yet.

    I mean all these set to "text bla bla", shouldn't they be saved in variables already, so you don't have to change that everytime...?
     
    LordDz_2, Dec 18, 2011
    #10
  11. ViroMan

    ViroMan Black Hole (*sniff*) Bully

    Messages:
    8,382
    Likes Received:
    4
    Trophy Points:
    0
    Shit... yep looks like crap.
     
    ViroMan, Dec 18, 2011
    #11
  12. w00kie

    w00kie Mustachioed Mexican

    Messages:
    3,863
    Likes Received:
    13
    Trophy Points:
    0
    omg :) looks like a funny way to protect your code.
     
    w00kie, Dec 18, 2011
    #12

Share This Page