Merge lp:~mcpeter/ubuntu-fr-forum/fix715133 into lp:ubuntu-fr-forum

Proposed by McPeter on 2011-02-20
Status: Merged
Merged at revision: 120
Proposed branch: lp:~mcpeter/ubuntu-fr-forum/fix715133
Merge into: lp:ubuntu-fr-forum
Diff against target: 248 lines (+69/-86)
6 files modified
.bzrignore (+5/-0)
BDD_update2.sql (+1/-1)
header.php (+1/-3)
include/template/main.tpl (+48/-68)
index.php (+13/-13)
lang/Francais/common.php (+1/-1)
To merge this branch: bzr merge lp:~mcpeter/ubuntu-fr-forum/fix715133
Reviewer Review Type Date Requested Status
YoBoY 2011-02-20 Needs Fixing on 2011-03-14
Review via email: mp+50503@code.launchpad.net

Description of the change

* Modification de fichiers
  - header.php
      * Modification du nom d'insertion css principal
      * Nettoyage du code

  - index.php
      * Nettoyage du code

  - lang/Francais/common.php
      * Modification du formulaire de login :
      * Retrait du code JS en dur (géré par common.js de static)
      * Ajout du code accessibilité
      * Ajout d'identifiant pour le JS (effets effacements au focus et
        input_focused)

  - include/templates/main.tpl
      * Ajout de l'accessibar
      * Nettoyage du code
      * Ajout du code accessibilité
      * Retrait du code JS en dur dans l'HTML (géré par common.js de static)

* Retrait du dossier STATIC (obsolète)

To post a comment you must log in.
McPeter (mcpeter) wrote :

Je suis en // entrain de "nettoyer" une partie du code dans le forum aussi ...
plein de portions de code commentées devenues inutiles et obsolètes.
Ainsi qu'une première simplification du code de reconnaissance du domaine (pas au point sur le forum :\ ) ... ça fait très "brouillon"

YoBoY (yoboy-leguesh) wrote :

Hum...
Vu que t'es lancé pourquoi tu n'as pas viré toutes les références à UFR_STATIC_DEV_TPL, d'ailleurs en enlevant le Define ça marche toujours ? il est dans le header.php je suppose ...

Je préfère pas savoir ce que fait ce pun_include site_domain.php... mais pourquoi ne pas avoir remplacer toutes les occurrences de ubuntu-fr.org comme dans l'entête ou le pied de page ?

review: Needs Fixing
McPeter (mcpeter) wrote :

c'est ce que je suis entrain de faire ... pour ça que j'en ai parlé dans mon commentaire suivant ... mais y'a des trucs de partout :\ .. .c'est l'horreur ... (et quand je pense que c'est en prod ça :'( )
je vois à appliquer la même formule que la tienne pour la doc.

YoBoY (yoboy-leguesh) wrote :

Me manque toujours la correction de quelques éléments pour que je merge, les liens qui renvoient vers ubuntu-fr.org uniquement au lien de renvoyer sur le domaine où on se trouve.

review: Needs Fixing
McPeter (mcpeter) wrote :

je suis dessus

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file '.bzrignore'
2--- .bzrignore 2009-06-03 19:43:27 +0000
3+++ .bzrignore 2011-02-20 13:31:40 +0000
4@@ -1,2 +1,7 @@
5 config.php
6 xcache-admin
7+cache/cache*
8+forum.dev.*
9+templates
10+toolbars
11+
12
13=== modified file 'BDD_update2.sql'
14--- BDD_update2.sql 2010-10-08 13:18:35 +0000
15+++ BDD_update2.sql 2011-02-20 13:31:40 +0000
16@@ -1,4 +1,4 @@
17-USE ubuntu_fr;
18+USE ubuntu_devforum;
19
20 -- phpMyAdmin SQL Dump
21 -- version 3.3.2deb1
22
23=== removed directory 'STATIC'
24=== modified file 'header.php'
25--- header.php 2010-10-21 22:17:41 +0000
26+++ header.php 2011-02-20 13:31:40 +0000
27@@ -99,8 +99,6 @@
28 ?>
29 <title><?php echo generate_page_title($page_title, $p) ?></title>
30 <?php
31- /* Ajout en dur pour la dev - McPeter - */
32- define('UFR_STATIC_DEV_TPL', 'http://www-static.ubuntu-fr.org/theme2010');
33 if(($pun_user['style'] == "Ubuntu")||($pun_user['style'] == "Ubuntu_menu-fixe")||(preg_match("/buntu_darker/",$pun_user['style'])))
34 {
35 $is_ubuntu = 0;
36@@ -161,7 +159,7 @@
37 // END ADD AlternativeSites
38
39 ?>
40- <link rel="stylesheet" media="screen" type="text/css" title="Design Ubuntu" href="<?=UFR_STATIC_DEV_TPL?>/css/forum-general.css" />
41+ <link rel="stylesheet" media="screen" type="text/css" title="Design Ubuntu" href="<?=UFR_STATIC_DEV_TPL?>/css/forum.css" />
42 <link rel="stylesheet" media="screen" type="text/css" title="Design Ubuntu" href="<?=UFR_STATIC_DEV_TPL?>/css/forum-<?=strtolower($distrib_name)?>.css" />
43
44 <?php }
45
46=== modified file 'include/template/main.tpl'
47--- include/template/main.tpl 2010-10-22 12:03:43 +0000
48+++ include/template/main.tpl 2011-02-20 13:31:40 +0000
49@@ -6,101 +6,81 @@
50 <pun_head>
51 </head>
52 <body>
53-<div id="accessibar">
54-</div>
55-<div id="pun<pun_page>">
56-<div id="page">
57- <!--<div id="header">
58- <div id="logo">
59- <pun_title>
60- <br /><a href="http://www.ubuntu-fr.org"><pun_desc></a>
61- </div>
62- <pun_status>
63- </div>-->
64- <div id="header">
65-&nbsp;
66+ <div id="accessibar">
67+ <a href="#main" tabindex="1">Contenu</a> | <a href="#qsearch__in" tabindex="2">Rechercher</a> | <a href="#navigation" tabindex="3">Menus</a>
68+ </div>
69+ <div id="pun<pun_page>">
70+ <div id="page">
71+
72+ <div id="header">
73+ &nbsp;
74 <div id="logo">
75 <h1>Ubuntu-fr</h1>
76 <a href="http://www.ubuntu-fr.org" title="Accueil">Communauté francophone d'utilisateurs <pun_include "site_de.php"><pun_include "site_nom.php"></a>
77-
78 </div>
79
80 <form action="search_ubuntufr.php" id="search">
81 <fieldset>
82- <!--
83- Modification McPeter - 27/06/2010 03:06
84-
85- <input type="text" value="Recherche rapide" name="search" size="34" /><select name="tsearch">
86- <option>Site</option>
87- <option>Documentation</option>
88- <option selected="selected" >Forum</option>
89- <option>Planet</option>
90-
91- </select>
92- -->
93- <!-- Copie de la doc -->
94- <input type="text" value="Recherche rapide..." onfocus="if(this.value == 'Recherche rapide...'){this.value='';} this.className='input_focused'" id="qsearch__in" accesskey="f" name="q" title="[F]" size="34" />
95+ <input type="text" value="Recherche rapide...." id="qsearch__in" accesskey="f" name="q" title="[F]" size="34" />
96 <select name="tsearch">
97 <option value="wiki">Documentation</option>
98 <option value="forum" selected="selected">Forum</option>
99 <option value="planet" >Planet</option>
100 </select>
101 <input type="submit" value="ok" class="button" title="Chercher" />
102- <!-- Fin modification -->
103 </fieldset>
104 </form>
105
106-<pun_status>
107+ <pun_status>
108
109-<div class="clearer"></div>
110+ <div class="clearer"></div>
111 </div>
112
113-
114-
115- <div id="navigation">
116+ <div id="navigation">
117 <ul>
118- <li class="menu"><a id="menu-accueil" class="title" href="http://www.<pun_include "site_domaine.php">.org"><span>Accueil</span></a></li>
119- <li class="menu"><a id="menu-doc" class="title" href="http://doc.<pun_include "site_domaine.php">.org" title="Documentation Ubuntu francophone"><span>Documentation</span></a></li>
120- <li class="menu" id="active" >
121- <a id="menu-forum" class="title" href="index.php" title="Index du forum"><span>Forum</span></a>
122- <ul>
123- <li class="cat"><h2>Navigation</h2>
124- <pun_navlinks></li>
125- <pun_searchlinks>
126- </ul>
127- </li>
128- <li class="menu"><a id="menu-planet" class="title" href="http://planet.<pun_include "site_domaine.php">.org"><span>Planet</span></a></li>
129+ <li class="menu"><a id="menu-accueil" class="title" href="http://www.<pun_include "site_domaine.php">.org"><span>Accueil</span></a></li>
130+ <li class="menu"><a id="menu-doc" class="title" href="http://doc.<pun_include "site_domaine.php">.org" title="Documentation Ubuntu francophone"><span>Documentation</span></a></li>
131+ <li class="menu" id="active" >
132+ <a id="menu-forum" class="title" href="index.php" title="Index du forum"><span>Forum</span></a>
133+ <ul>
134+ <li class="cat">
135+ <h2>Navigation</h2>
136+ <pun_navlinks>
137+ </li>
138+ <pun_searchlinks>
139+ </ul>
140+ </li>
141+ <li class="menu"><a id="menu-planet" class="title" href="http://planet.<pun_include "site_domaine.php">.org"><span>Planet</span></a></li>
142 </ul>
143 <div class="clearer"></div>
144- </div>
145+ </div>
146
147-<div id="wrapper">
148+ <div id="wrapper">
149 <pun_announcement>
150 <div id="main">
151- <div id="hidemenu"><a accesskey="z" title="Masquer le menu - AccessKey Z" href="javascript:switchMenuVisible()"><span>&nbsp;</span></a></div>
152- <pun_main>
153+ <div id="hidemenu" title="Masquer le menu"></div><!--
154+ <div id="hidemenu"><a accesskey="z" title="Masquer le menu - AccessKey Z" href="javascript:switchMenuVisible()"><span>&nbsp;</span></a></div>-->
155+ <pun_main>
156 </div>
157-
158+
159 <div id="footer">
160-
161- <pun_footer>
162-
163- <div class="end-box"><div><!-- Bottom corners --></div></div>
164- </div>
165+ <pun_footer>
166+ <div class="end-box"><div><!-- Bottom corners --></div></div>
167+ </div>
168+ </div>
169+
170+ <div class="block-inner">
171+
172+ <div class="content">
173+ <ul class="menu"><li class="leaf first contact"><a href="http://www.ubuntu-fr.org/contacts">Contact</a></li>
174+ <li class="leaf active-trail ressources"><a class="barreverticale active" href="http://www.ubuntu-fr.org//">Ressources</a></li>
175+ <li class="leaf last licences"><a class="barreverticale" href="http://www.ubuntu-fr.org/licences">Licences</a></li>
176+ </ul>
177+ </div>
178+
179+ <div class="clearer"></div>
180+ </div>
181 </div>
182-
183- <div class="block-inner">
184-
185-
186- <div class="content">
187- <ul class="menu"><li class="leaf first contact"><a href="http://www.ubuntu-fr.org/contacts">Contact</a></li>
188-<li class="leaf active-trail ressources"><a class="barreverticale active" href="http://www.ubuntu-fr.org//">Ressources</a></li>
189-<li class="leaf last licences"><a class="barreverticale" href="http://www.ubuntu-fr.org/licences">Licences</a></li>
190-</ul> </div>
191-
192- <div class="clearer"></div>
193-
194- </div> </div>
195-</div>
196+ </div>
197 </body>
198 </html>
199
200=== modified file 'index.php'
201--- index.php 2010-11-16 09:51:12 +0000
202+++ index.php 2011-02-20 13:31:40 +0000
203@@ -40,19 +40,19 @@
204 require PUN_ROOT.'header.php';
205
206 ?>
207- <div id="navbar">
208-<div class="postlinkst">
209- <div class="inbox crumbsplus">
210- <ul class="crumbs">
211- <li><a href="http://www.ubuntu-fr.org">Accueil</a></li>
212- <li><span>&raquo;&#160;</span><strong><?php echo $lang_common['Index'] ?></strong></li>
213- </ul>
214- <p class="pagelink conl"><?php echo $paging_links ?></p>
215-<?php echo $post_link ?>
216-<?php echo $subscraction ?>
217- <div class="clearer"></div>
218- </div>
219-</div>
220+<div id="navbar">
221+ <div class="postlinkst">
222+ <div class="inbox crumbsplus">
223+ <ul class="crumbs">
224+ <li><a href="http://www.ubuntu-fr.org">Accueil</a></li>
225+ <li><span>&raquo;&#160;</span><strong><?php echo $lang_common['Index'] ?></strong></li>
226+ </ul>
227+ <p class="pagelink conl"><?php echo $paging_links ?></p>
228+ <?php echo $post_link ?>
229+ <?php echo $subscraction ?>
230+ <div class="clearer"></div>
231+ </div>
232+ </div>
233 </div> <!-- fin du block navbar, voir le template-->
234
235 <!-- Begin Forum accueil + gagnant du mois-->
236
237=== modified file 'lang/Francais/common.php'
238--- lang/Francais/common.php 2010-10-21 22:17:41 +0000
239+++ lang/Francais/common.php 2011-02-20 13:31:40 +0000
240@@ -99,7 +99,7 @@
241 <fieldset>
242 <input type="hidden" name="form_sent" value="1" />
243 <input type="hidden" name="redirect_url" value="index.php" />
244- <input type="text" value="Identifiant" onfocus="if(this.value == \'Identifiant\'){this.value=\'\';} this.className=\'input_focused\'" name="req_username" size="25" maxlength="25" tabindex="1" /><input type="password" value="Mot de passe" onfocus="if(this.value == \'Mot de passe\'){this.value=\'\';} this.className=\'input_focused\'" name="req_password" size="25" tabindex="2" /><input type="submit" value="connexion" /> / <a href="register.php">inscription</a>
245+ <label for="u_field">Identifiant : </label><input type="text" value="Identifiant" name="req_username" id="u_field" size="9" /><label for="p_field">Mot de passe : </label><input type="password" value="Mot de passe" name="req_password" id="p_field" size="9" alt="Mot de passe" /><input type="submit" value="connexion" id="connect" /> / <a href="register.php">inscription</a>
246 </fieldset>
247 </form>',
248 'Profile' => 'Profil',

Subscribers

People subscribed via source and target branches

to all changes: