Skip to content

Commit

Permalink
Admin: Move news block after users and courses
Browse files Browse the repository at this point in the history
  • Loading branch information
@AngelFQC
AngelFQC committed Mar 6, 2019
1 parent b12027f commit df34cc9
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 17 deletions.
2 changes: 1 addition & 1 deletion app/Resources/public/css/base.css
View file
Expand Up @@ -9069,7 +9069,7 @@ input.form-control[type="color"] {
clear: left;
}

#settings .settings-block:nth-child(2n+1) {
#settings .settings-block:nth-child(2n+2) {
clear: left;
}
}
Expand Down
32 changes: 16 additions & 16 deletions main/template/default/admin/settings_index.tpl
View file
Expand Up @@ -54,26 +54,11 @@
});
</script>

{% if not admin_chamilo_announcements_disable %}
<section id="chamilo-news" class="row hidden">
<div class="col-xs-12">
<div class="alert alert-info">
<button type="button" class="close" data-dismiss="alert" aria-label="Close">
<span aria-hidden="true">&times;</span>
</button>
<div id="chamilo-news-content">

</div>
</div>
</div>
</section>
{% endif %}

<section id="settings" class="row">
{% set columns = 2 %}
{% for block_item in blocks %}
{% if block_item.items %}
<div id="tabs-{{ loop.index }}" class="settings-block col-md-6">
<div id="tabs-{{ loop.index }}" class="{{ loop.index > 2 ? 'settings-block' : '' }} col-md-6">
<div class="panel panel-default {{ block_item.class }}">
<div class="panel-heading">
{{ block_item.icon }} {{ block_item.label }}
Expand Down Expand Up @@ -116,6 +101,21 @@
</div>
</div>
{% endif %}

{% if loop.index == 2 and not admin_chamilo_announcements_disable %}
<section id="chamilo-news" class="col-md-12 hidden">
{#<div class="col-xs-12">#}
<div class="alert alert-info">
<button type="button" class="close" data-dismiss="alert" aria-label="Close">
<span aria-hidden="true">&times;</span>
</button>
<div id="chamilo-news-content">

</div>
</div>
{#</div>#}
</section>
{% endif %}
{% endfor %}
</section>

Expand Down

17 comments on commit df34cc9

@NicoDucou
Copy link
Member

Choose a reason for hiding this comment

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

Iba a reportar un error con la presentacion del annucio porque de pronto lo vi debajo de user y course
pero parece que es un feature entonces lo anoto aqui porseacaso y saber si es realmente normal que se vea asi :
image

@ywarnier
Copy link
Member

Choose a reason for hiding this comment

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

sí, es normal. Arriba molestaba porque descuadraba los primeros bloques después de mostrarlos. Moviéndolos una fila a bajo molesta mucho menos pero sigue visible

@aragonc
Copy link
Member

@aragonc aragonc commented on df34cc9 Mar 7, 2019

Choose a reason for hiding this comment

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

pikachu-meme

@jmontoyaa
Copy link
Member

@jmontoyaa jmontoyaa commented on df34cc9 Mar 7, 2019 •
edited

Choose a reason for hiding this comment

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

A mi me molesta más tenerlo en el medio de la página ... 😅

@ywarnier
Copy link
Member

Choose a reason for hiding this comment

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

Por?

@jmontoyaa
Copy link
Member

Choose a reason for hiding this comment

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

Es subjetivo :)

@ywarnier
Copy link
Member

Choose a reason for hiding this comment

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

Sí, depende de las herramientas que más vas a usar, probablemente. Si usas más sesiones o el bloque de plataforma, entonces el hecho que "salte" un poco cuando aparece el bloque es molestoso, pero considero que en la mayoría de los casos los admins van a ir a la página de administración primero para trabajar sobre los usuarios o los cursos. No tenemos suficiente información para saber esto a ciencia cierta, pero es lo que veo cuando otros usan el panel de administración.
Otra solución es pasarlo después de los bloques plataforma y sesión, pero ahí ya no lo verían y es el punto aquí: es que lo vean, porque necesitamos más recursos para competir en el mundo de los LMSes y mucha gente no sabe que hay servicios disponibles de parte de empresas profesionales.

@aragonc
Copy link
Member

@aragonc aragonc commented on df34cc9 Mar 7, 2019

Choose a reason for hiding this comment

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

Estaba mejor en la primera posición como antes, porque se cambio?

@ywarnier
Copy link
Member

Choose a reason for hiding this comment

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

Porque antes cargabas la página, te alistabas para dar clic en "Lista de usuarios" y ahí saltaba para mostrarte el mensaje y terminabas dando clic justo en el enlace de proveedores en lugar de ir a la lista de usuarios, lo cual era más molestoso.

@ywarnier
Copy link
Member

Choose a reason for hiding this comment

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

Si encontramos una mejor manera, estoy abierto a cambios. El problema actual es que es un AJAX que solo muestra el cuadro azul cuando llega a recuperar datos del servidor de Chamilo.
Si llegamos a que los primeros bloques no "salten" tan violentamente hacia abajo o que podamos reservar un espacio que se llene con otra cosa (alguna imagen bonita y libre) si es que no recibimos respuesta en AJAX, estoy de acuerdo...

@AngelFQC
Copy link
Member Author

Choose a reason for hiding this comment

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

¿Y si queda flotando sobre la parte inferior de la página?

@ywarnier
Copy link
Member

@ywarnier ywarnier commented on df34cc9 Mar 7, 2019 •
edited

Choose a reason for hiding this comment

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

Porque no?... habría que probarlo y que no impida usar las opciones de la última parte de la página cuando llegas ahí.

@ywarnier
Copy link
Member

Choose a reason for hiding this comment

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

Lo pruebas, @AngelFQC ?

@AngelFQC
Copy link
Member Author

Choose a reason for hiding this comment

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

Cambio hecho en d49ee90

@NicoDucou
Copy link
Member

Choose a reason for hiding this comment

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

Esta muy bien asi.

@jmontoyaa
Copy link
Member

Choose a reason for hiding this comment

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

Queda mejor ahora, bien!

@ywarnier
Copy link
Member

Choose a reason for hiding this comment

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

Sí, me va mejor también. Gracias.

Please sign in to comment.