Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Legislation process HTML #1704

Closed
decabeza opened this issue Jun 26, 2017 · 4 comments
Closed

Legislation process HTML #1704

decabeza opened this issue Jun 26, 2017 · 4 comments
Milestone

Comments

@decabeza
Copy link
Collaborator

Background

When a legislation process have a draft version published and the allegations phase is enabled you can go to /legislation/processes/N/draft_versions/N

What

There is two minor problems to fix in this view:

  • There is a duplicated id="debate-show" (used to interact with JS), this generates an HTML validation error.

  • When you clic on arrow icon "View process information" appears/disappears several icon-angle-up icons.

before clic
view_process_info

after clic
view_process_info_2

@xarlybovi
Copy link
Contributor

I can fix it

@iagirre
Copy link

iagirre commented Jan 2, 2018

Is something left to do here?

@decabeza
Copy link
Collaborator Author

decabeza commented Jan 2, 2018

Hi @iagirre we have a pending PR to review (#1899) but I think it's already done ;)

@voodoorai2000 voodoorai2000 moved this from Next to Design in Roadmap Jan 3, 2018
@voodoorai2000 voodoorai2000 moved this from Design to Doing in Roadmap Jan 3, 2018
@voodoorai2000 voodoorai2000 moved this from Doing to Done in Roadmap Jan 3, 2018
@bertocq bertocq added this to the v0.13 milestone Jan 4, 2018
@MariaCheca MariaCheca moved this from Done to Review in Roadmap Jan 8, 2018
@voodoorai2000 voodoorai2000 moved this from Review to Done in Roadmap Jan 8, 2018
@voodoorai2000
Copy link
Member

Closed via #1899

@voodoorai2000 voodoorai2000 removed this from Done in Roadmap Jan 22, 2018
javierm added a commit to javierm/consul that referenced this issue Dec 18, 2018
…mize_toda_la_ciudad

Move "Toda la ciudad" code to custom files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants