Skip to content

Commit

Permalink
navettes: hide Senate/AN link if it doesn't exist yet
Browse files Browse the repository at this point in the history
  • Loading branch information
@mdamien
mdamien committed Sep 25, 2018
1 parent ee838f7 commit b11b7e5
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions public/modules/navettes/navettes.viz.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -1039,8 +1039,12 @@ reset_filters();

extrainfo += '<p class="sources">';
extrainfo += '<small>';
extrainfo += '<a href="' + d.url_dossier_senat + '" target="_blank"><span class="glyphicon glyphicon-link"></span> dossier Sénat</a> &nbsp; &nbsp; ';
extrainfo += '<a href="' + d.url_dossier_assemblee + '" target="_blank"><span class="glyphicon glyphicon-link"></span> dossier Assemblée</a>';
if (d.url_dossier_senat) {
extrainfo += '<a href="' + d.url_dossier_senat + '" target="_blank"><span class="glyphicon glyphicon-link"></span> dossier Sénat</a> &nbsp; &nbsp; ';
}
if (d.url_dossier_assemblee) {
extrainfo += '<a href="' + d.url_dossier_assemblee + '" target="_blank"><span class="glyphicon glyphicon-link"></span> dossier Assemblée</a>';
}
extrainfo += d.url_jo ? '<br/><a href="' + d.url_jo + '" target="_blank"><span class="glyphicon glyphicon-link"></span> loi sur Légifrance</a>' : '';
extrainfo += ' &nbsp; &nbsp; <a href="' + APIRootUrl + d.id + '/" target="_blank"><span class="glyphicon glyphicon-link"></span> OpenData</a>';
//extrainfo += ' / <a href="http://git.lafabriquedelaloi.fr/parlement/' + d.id + '/" target="_blank">Git</a>';
Expand Down

3 comments on commit b11b7e5

@boogheta
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai l'impression que dans ces cas là ça fait que certains ne sont plus à leur place normale non ? Il faut s'assurer qu'ils ne bougent pas en remplissant des placeholder je pense (mais je n'ai pas testé donc si ça se trouve pas de souci en fait ;) )

@mdamien
Copy link
Member Author

@mdamien mdamien commented on b11b7e5 Oct 15, 2018 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Le seul saut à la ligne que je vois est pour l'url JO, ce saut disparaît mais le résultat est pas mal:

image

EDIT: Oups, j'ai mal compris, c'est pour la vue articles OK

@boogheta
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cf #234

Please sign in to comment.