Giter Site home page Giter Site logo

Comments (6)

babaissarkar avatar babaissarkar commented on June 8, 2024

If we're consolidating those dialogs, I'd suggest the following two things :

  1. It should deal with #1059
  2. Putting something as important as Recruit/Recall into a right click context menu doesn't feel good. One idea would be to show the recruit/recall when mouse hovers on top of a castle tile while the leader is in the keep. Haven't thought this in detail, but it'd atleast save a lot of clicks and make it obvious. (I've even thought of a design, might post that later.)

from wesnoth.

CelticMinstrel avatar CelticMinstrel commented on June 8, 2024

One idea would be to show the recruit/recall when mouse hovers on top of a castle tile while the leader is in the keep.

I'm not entirely sure what you mean by this, but I think it should be obvious that opening a modal dialog in response to a mouse hover is a very bad idea. Maybe that's not what you meant though.

Also, I'm not quite sure but it sounds like your point 2 is likely out of scope for this issue.

from wesnoth.

gfgtdf avatar gfgtdf commented on June 8, 2024

the main thing would be to expose the recruit dialog, so that SotA can handle its corpse recruits in a more reasonable manner

If thats what you want to do, you'd need to first add a way for lua to hook in before the original dialog is shown (to be replaced with your custom one), which is imo the more complicated part

from wesnoth.

CelticMinstrel avatar CelticMinstrel commented on June 8, 2024

which is imo the more complicated part

What's complicated about that, exactly…? Just find the place that currently calls the dialog and do something else instead, right?

Not entirely sure that's actually necessary though.

from wesnoth.

babaissarkar avatar babaissarkar commented on June 8, 2024

I'm not entirely sure what you mean by this, but I think it should be obvious that opening a modal dialog in response to a mouse hover is a very bad idea. Maybe that's not what you meant though.

That's right. Is it possible with a modeless dialog (assuming we can get it working)?

Also, I'm not quite sure but it sounds like your point 2 is likely out of scope for this issue.

Could be, perhaps there should be another issue on consolidating the three dialogs the you mentioned?

from wesnoth.

CelticMinstrel avatar CelticMinstrel commented on June 8, 2024

Is it possible with a modeless dialog (assuming we can get it working)?

Maybe, but that sounds like a strange way to do it…

from wesnoth.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.