Circuitverse: Migrate tooltips

Created on 27 Feb 2019  路  17Comments  路  Source: CircuitVerse/CircuitVerse

Currently, tooltips information are stored at UX.js, migrate the same to individual circuit elements. Check out how Power and Ground implement Tooltips.

Most helpful comment

@armudgal I think you tagged the wrong person :sweat_smile:

All 17 comments

I would like to work on this issue.

Hi @rkpattnaik780, can you please if you are working on this issue

Hi I would like to take up this issue if not taken

Hey @satu0king !
If no one is working on this issue, than I am willing to work on it!

@sarthak77 Would you be taking up this issue?

ok i'll be working on it

Hi ~@satu0king~, @sarthak77 the issues are based on first come first serve, so sorry for that, please look through other issues. Cheers

Just realised I tagged Satvik by mistake.

@armudgal I think you tagged the wrong person :sweat_smile:

Currently, tooltips information are stored at UX.js, migrate the same to individual circuit elements. Check out how Power and Ground implement Tooltips.

Hey I could not relate to what do you mean by Power and Ground
Can you elaborate?

Currently, tooltips information are stored at UX.js, migrate the same to individual circuit elements. Check out how Power and Ground implement Tooltips.

Hey I could not relate to what do you mean by Power and Ground
Can you elaborate?

Look for this line in modules.js:
Power.prototype.tooltipText = "Power: All bits are High(1).";

This is how we want all modules to implement tooltips now.

@sarthak77 since we haven't heard from you, I am releasing this issue. Please feel free to claim the issue again if you are still working on this.

I was having trouble in setting up the system and had exams in between so was out for a while

i want to give this issue a try, can i take it?

This is a very easy issue, therefore the first PR will be merged.

@prajapatgautam1999 You can go ahead since we didn't receive any PR till now

migration done under pull request #286

@prajapatgautam1999 no need to do so i already fixed the issue in pr #286

Was this page helpful?
0 / 5 - 0 ratings

Related issues

tachyons picture tachyons  路  6Comments

darkshredder picture darkshredder  路  8Comments

ryuk156 picture ryuk156  路  9Comments

tachyons picture tachyons  路  6Comments

Ananthavijay picture Ananthavijay  路  5Comments