Acción - tabs 16

+

Mover la acción de las tareas locales tareas locales que han rendido como se pestañas. Las pestañas son principalmente un patrón para seperating diferentes puntos de vista '' en los datos, diferentes espacios de información. En este momento, también estamos poniendo "acciones" de allí, las cosas que deberían ser botones o enlaces. Además de ser fácilmente pasados ​​por alto, estamos mezclando dos "vista" y las operaciones de "acción" en las tareas locales y necesitamos un mecanismo que trata a los dos como las cosas que son separadas. Pestañas para las pestañas, enlaces y botones para las acciones. Pestañas en la parte superior, las acciones justo allí donde los necesite, en el contexto del elemento (s) que realizan una acción por (que probablemente significa: libre para ser colocado donde sea necesario). efecto secundario muy negativo: botones arbitrarias son (más probable?) no figuran en el sistema de menús? ¿Cómo puede admin_menu exponer esas páginas? Grande, este concepto se aplica a los nodos (o contenido). Pero ¿qué pasa con contrib módulos para proporcionar el contenido o no-tan-realmente-páginas de contenido? Se supone que acaba de poner sus botones "en algún lugar"? Si esa es la supuesta dirección, entonces esto llevará a una interfaz de usuario muy inconsistente * *. Bojhan Atribución de crédito: Bojhan comentó 8 agosto 2009 a las 9:43 am Soy consciente del problema contrib, se supone que deben poner sus acciones también como botones. Pero este escenario no se adapta bien a muchas acciones. No debe conducir a una interfaz inconsistente, si todo el mundo se divide acciones de los listados. yoroy Atribución de crédito: yoroy comentó 9 agosto 2009 a las 24:47 Olvidar el 'se puede colocar en cualquier lugar ". Por ahora. opbjective principal es la necesidad de dividir la representación local_task en dos, uno para los enlaces de la página / de la lista, uno para las acciones / botones. El argumento de la coherencia que aquí no hace mucho sentido. No guardar mi martillo y destornilladores en las estanterías, aunque eso sería consisten en cómo almaceno mis libros ... Necesitamos un nuevo lugar para las herramientas en local_tasks. Creo que esto tiene tareas locales identifiying como uno o el otro y un valor predeterminado para el escenario definido. Gábor Hojtsy Atribución de crédito: Gábor Hojtsy comentó 10 agosto 2009 a las 7:54 am sistema de menús: así, como dice @yoroy, podemos hacer que estos elementos de menú; acaba de añadir otro tipo de elemento de menú, además de las tareas locales, por ejemplo. "acciones locales"? O cambiar el nombre de las tareas locales de "pestañas locales" y añadir "acciones locales", por lo que de alguna manera comunicamos la idea de que las tareas locales deben ser utilizados para diferentes puntos de vista y no para la navegación. Consistencia: Mi entendimiento es que tenemos pantallas CRUD, donde se tiene una lista de cosas (usuarios, productos, nodos, términos de la taxonomía, vocabularios, etc.). Puede configurar o eliminarlos de las operaciones allí y usted debe tener su Crear un enlace allí mismo en la parte superior. adjunto parche es un corte total, pero algunas ideas podría ser posible llevar en una fina aplicación. 1. Se ha añadido una nueva bandera de fichas: MENU_IS_LOCAL_ACTION. 2. Filtro Añadido a menu_local_tasks () para filtrar a esto. (Añadido este filtro para su clave de caché interna). 3. Agregado prestación de estos en la plantilla en siete (no hay otros temas aún). 4. Aplicado algún artilugio a la CSS, de modo que el icono + muestra siempre. 5. Modificado dos rutas de módulo menú para utilizar esto para la acción Agregar. Un par de problemas / preguntas con este enfoque: - La acción muestra en todas las rutas en el mismo nivel (debido a la forma en que es una ficha después de todo), que es sin duda no desea - no debe mostrar en hermanos pestañas a la pestaña principal visión general o en la página sumarse - iconos distintos no son compatibles con acciones distintas - ¿qué tipo de otras acciones debemos apoyar adicionalmente aquí para agregar? - No hay interpretación de los enlaces de acción con el texto de ayuda; deben mostrar enlaces de acción por encima o por debajo de la ayuda? (El parche les muestra a continuación, el tipo de las oscurece a un degreee) Por lo que el parche añade el menú Agregar y agregue las acciones de enlace como se puede ver aquí: Bojhan Atribución de crédito: Bojhan comentó 13 de agosto de 2009 a 15:36 - Muestra la acción en todas las rutas en el mismo nivel (debido a la forma en que es una ficha después de todo), que es sin duda no desea - no debe mostrar en hermanos pestañas a la pestaña principal visión general o en la página sumarse Eso no es deseable en efecto je, no se puede hacer la función se comporta bien, aunque? - iconos distintos no son compatibles con acciones distintas - ¿qué tipo de otras acciones debemos apoyar adicionalmente aquí para agregar? Quizás editar? Por ejemplo, en estilos de imagen, para editar el título. - No hay interpretación de los enlaces de acción con el texto de ayuda; deben mostrar enlaces de acción por encima o por debajo de la ayuda? (El parche les muestra a continuación, el tipo de las oscurece a un degreee) A continuación, que son parte del contenido, no forma parte del título. Creo que mira algo oscura es un problema de separación. sol de crédito Reconocimiento: sol comentó 17 de agosto de 2009 a 08:28 am Eso sería en realidad un mapa a # 460320: Normalizado, conectable entidad de carga (nodos, usuarios, taxonomía, archivos, comentarios) - definición de entidades en el código, junto con sus pestañas / tareas / whateverYouNameThem. enrutador menú (node_menu ()) se convierte en y todas por debajo se desplaza a la entidad. Editar un campo? Tráeme las tareas locales para el campo%. Editar un bloque? Tráeme las tareas de los bloques. Editar un nodo? Tráeme las tareas para el nodo%. etc. pp. El mismo se aplica a la separación de fichas de acciones de tareas. (Cualesquiera que sean los "botones" son en este idioma) tareas locales para menu_get_object () se derivan de la entidad, rendido como las tareas locales. Además de los otros elementos del sub-enrutador definidos a través de hook_menu (). "Botones" utilizan una especial MENU_ * "tipo", lo que los hace '_visible' = FALSO por defecto, por lo que no aparecen en el árbol de menús, ni como tareas locales. admin_menu se siente libre para cambiarlos, por lo que aparecen en el árbol de menús. La lógica para tareas locales mantiene el mismo, sin tener en cuenta la propiedad '_visible'. Ok, aquí es un parche con un enfoque actualizado. He utilizado los elementos de menú se asemejan a los elementos de menú normales, por lo que se mostrará en los menús por defecto. Esto puede no ser lo que queremos (en cuyo caso debemos exigir que todos los temas para apoyar esta área de acción), pero necesitamos alguna información sobre que supongo. Así que he introducido un nuevo tipo de elemento de menú llamado MENU_LOCAL_ACTION. Que tiene una función independiente para tomar las acciones del niño para la página actual. Entonces convertido los dos elementos del menú de administración (que resultó ser sorprendentemente buenos conejillos de indias), y se aplican los mismos cambios en el tema como en el parche anterior. Ahora bien, esto toma acciones locales muy bien en las páginas, en el que el padre no es un elemento comodín (% no tiene algo), pero siempre y cuando% algo está en la ruta de enrutador, la mesa menu_links deja de dar información de los padres, por lo que necesitaría algún otro tipo de artimañas para hacer este trabajo. De todos modos, esto sería de hecho a cobro revertido acciones relacionadas en virtud de un área de acción como @sun explicó, por lo que sería fácil de conseguir acciones locales en el menú, incluso en comparación con intentar quedarse con la recogerlos desde las fichas que contienen relacionados y otros puntos de vista de la el contenido en cuestión. Me encantaría retroalimentación concepto y específicamente el menú de ayuda en la crianza de los hijos que no funciona con todo para la página de menú de la lista. (@todo En el código). Captura de pantalla todavía que el anterior (para la página del menú principal). pwolanin Atribución de crédito: pwolanin comentó 17 de agosto de 2009 a 19:49 ¿Por qué necesitamos el nuevo tipo? Vamos a hacer ellos MENU_NORMAL_ITEM y ponerlos manualmente en la página cuando sea necesario. @Gabor - El "elemento de menú actual" es un elemento de router. Tenemos un mecanismo bastante desarrollada para las tareas locales - Creo que si es realmente imprescindible disponer de una forma automatizada para poner presentes "tareas" a continuación, las tareas deben ser una variante de un MENU_LOCAL_TASK. Además, me gustaría prioridad a esta igual o mayor: # 353208: Permitir a las tareas locales en tiempo de ejecución condicional (hook_menu_local_tasks_alter ()) @Damien - La cuestión aquí es algo diferente - ya podemos rellenar las partes para las tareas locales basadas en el camino. Gábor Hojtsy Atribución de crédito: Gábor Hojtsy comentó 18 de agosto de 2009 a 24:47 @pwolanin: elementos de acción no son como las tareas locales en que no hay una acción por defecto y las acciones locales sólo se muestran justo debajo de sus padres, a diferencia de las pestañas que se muestran con todos los hermanos. Así que las acciones locales son mucho más como elementos de menú normales en comparación con las tareas locales. Por supuesto, si usted dice independientemente de su aparente falta de comportamiento tarea local (véase el parche anterior por encima de lo que sucede si aplicamos la lógica tarea local para los que he intentado antes) aún debemos hacer que las tareas locales porque no es especial carcasa de la crianza de código para tareas locales, también podemos ir por ese camino, pero entonces nosotros puede doblar hacia que este comportamiento, así, lo que hace aún más especial en algunos aspectos. Como se ha dicho, lo he intentado este camino anteriormente, la adición de un marcador poco a las tareas locales en lugar de hacer estos artículos más a las tareas locales, pero a medida que los programas de demostración, tratándolos como tareas locales no es la experiencia de interfaz de usuario que queramos. pwolanin Atribución de crédito: pwolanin comentó 18 de agosto de 2009 a 13:23 Bueno, mirando a esta implementación actual api. drupal. org/api/function/menu_local_tasks/7~~V~~singular~~3rd también es triste que nos vamos a volver los datos convertidos en comparación con los datos estructurados. Debemos arreglar eso también. Gábor Hojtsy Atribución de crédito: Gábor Hojtsy comentó 18 de agosto de 2009 a 13:25 @pwolanin: el código se inspiró en el código de representación pestaña que hace lo mismo. Vamos a obtener una dirección para la implementación interna primero que diría! rickvug Atribución de crédito: rickvug comentado 19 de agosto de 2009 a 07:46 am Dios mío, sí! La mejora de la interfaz de usuario esto permitiría es muy necesaria. Veo este ajuste, una guía clara para contrib, la mejora de la coherencia global de interfaz y en última instancia, mejorar UX. Hubo mucho debate acerca de esta distinción en el # 196758: Tener borrar como un botón en el formulario de edición de nodo significa que ciertos usuarios no tienen acceso a ella cuando deberían. en última instancia, la cuestión se cale. Yo y otros argumentaron en contra de hacer borrar una tarea local específicamente porque pestaña estilo sólo se debe utilizar para diferentes vistas de datos. Veo este problema como un bloqueador para mover borrar (si eso es incluso una buena idea en primer lugar es otra historia). Añadido $ local_actions a la lista de variables, documentada en page. tpl por defecto. Garland y Stark tienen un tratamiento de lista por defecto para los enlaces. No estoy del todo convencido de que este es el camino que debemos hacerlo en Garland, pero Stark es un tema de arranque, por lo que podría no ser malo esta manera (a pesar de que las pestañas tienen sus estilos). Se ve en Garland: Además, Mark tuvo una discusión bastante larga con la gente de la comunidad UX y resultó que no puede haber un tipo diverso de acciones, por lo que compensando los pequeños iconos especiales para ellos no es extensible. Así que en lugar de ir hacia adelante con el icono de los enlaces, se sugirió que ir con los botones. Esta revisión actualizada incluye también que el cambio visual. Nuevas miradas en Seven: yoroy Atribución de crédito: yoroy comentó 21 de agosto de 2009 a 18:18 Anterior dirección de enlace de texto estaba bien. Estos botones son muy incómodo mirando en esta posición por encima del contenido principal de una página. No es que habrá 10 tipos diferentes de acciones. Actualmente contamos con 1 punto en el núcleo que muestra las acciones 2, que es en los foros, donde se puede añadir un foro de discusión y un recipiente foro. Así, tanto las variaciones en añadir, incluso. Yo diría, mantener la atención en la separación real de las acciones de las pestañas. El rediseño de la mirada de las acciones está mejor servido en un seguimiento, por ahora, por favor volver a los enlaces de texto. Bojhan Atribución de crédito: Bojhan comentó 21 de agosto de 2009 a 18:28 Creo que la principal preocupación era, que los enlaces de tipo de configuración - ya sea que mover un nivel hacia arriba o se convierten en acciones. Que es factible para cosas como el menú de edición, editar vocabulario - para desplazarse hacia arriba. Pero en la parte superior, digamos / menú / configure. Esto no es posible, como se ha señalado yoroy las acciones como botones tienen un aspecto un tanto incómoda. Estoy muy bien con la comisión sólo los enlaces de texto, y tratando de encontrar la manera de manejar las otras acciones en los seguimientos. O incluso la apariencia de acciones. Se seca Atribución de crédito: Dries comentaron 22 de agosto de 2009 a 19:32 Por el amor de DX, sigo pensando que es útil para cambiar el nombre MENU_DEFAULT_LOCAL_TASK a otra cosa; TASK y ACCIÓN sienten que están demasiado cerca de otro, y por lo tanto potencialmente confusa. Yo sugeriría cambiar el nombre de xxx_TASK a xxx_TAB. Podemos hacer esto en un parche de seguimiento, si se desea. De lo contrario, este parche se ve 100% listo. Otras cosas que podrían ser fijos, ya sea en este parche o parches de seguimiento: * "Añadir nuevo contenido" en la? Q = admin / contenido * "Añadir usuario" en la? Q = admin / persona * "Añadir bloque" en la? Q = admin / estructura / bloque * "Añadir tipo de contenido" en la? Q = admin / estructura / tipos * "Añadir menú" en la? q = admin / estructura / menú * "Añadir vocabulario" en la? q = admin / estructura / taxonomía * "Añadir estilo" en la? q = admin / config / media / imagen-styles sol de crédito Reconocimiento: sol comentó 22 de agosto de 2009 a 19:39 El cambio de nombre de los internos define Me gustaría aplazar a un parche de seguimiento, pero lo ideal a otro problema, porque es muy técnico. En este tema, quiero enviar parches de seguimiento para convertir las acciones restantes como usted dijo, por lo que el equipo puede revisar UX + foco en aquellos. Yo ya empecé con ese trabajo. Estoy de acuerdo con Dries que la distinción entre una "tarea" y una "acción" será un total de WTF para nuevos desarrolladores. Pero podía ver entrar en este territorio bikeshed, por lo que de acuerdo con empujándolo fuera a otro tema. También me trajo hasta que eso es feo como el pecado, y sólo debería ser de impresión $ action_links; como el que tenemos la impresión $ primary_links y tal. Pero el sol señaló que esto sólo es conforme al código existente. Lo suficientemente justo. Desde la conversión tarea requiere este andamiaje en su lugar, y se seca parece estar bien con ella aparte de estas dos cosas, me comprometí a HEAD. Necesidades en la documentación de la guía de actualización del módulo. Se seca Atribución de crédito: Dries comentaron 22 de agosto de 2009 a 20:00 @sun, suena muy bien! He cometido el parche en el # 41, de modo que podemos centrarnos en los parches de seguimiento. ¡Gracias! sol de crédito Reconocimiento: sol comentó 22 de agosto de 2009 a 20:34 »Revisado probado por la comunidad ja. Obviamente, algo salió mal: - D drupal. org/cvs? commit=254832 Parece que tenemos 2 nuevas características distintas ahora;) No estoy seguro de lo que esta confirmación cuestión pertenecía. Este parche se convierte, además, todas las tareas locales que se seca mencionan anteriormente. Sin embargo: Acción enlaces no aparecen en admin / estructura / bloque, ni sobre admin / estructura / block / lista, ni sobre admin / estructura / block / lista / siete. He tratado de arreglar eso, pero terminó en pesadilla sistema de menús. Voy a crear un tema aparte para que (tal vez mezclado con el cambio de nombre define), debido a que pueden requerir bastante reescritura de menu_local_tasks () para las acciones. Bojhan Atribución de crédito: Bojhan comentó 22 de agosto de 2009 a 23:31 »Revisado probado por la comunidad Marcada, y se actualiza todas las páginas que pude ver. webchick Atribución de crédito: webchick comentó 22 de agosto de 2009 a 23:44 Comentado probado por la comunidad En IRC Bojhan está señalando otros lugares que no son convertidas, por lo golpeando fuera de mi cola. ¡DECIR AH! ;) errar. Tengo dificultades para entender lo que se requiere para obtener este parche en. El gancho está documentado ya no inventado aquí. $ data quizás podrían ser renombrados, pero que no pertenece a este tema ni parche. Este parche solo implementa un gancho existente en algunos lugares. No estoy seguro de si explicamos cada aplicación solo gancho. ¿Puede alguien explicarme qué es exactamente falta? coger Atribución de crédito: captura comentado 3 de diciembre 2009 a las 4:31 am Lo haría "// Añadir" Añadir nuevo contenido "como la acción local. Utilizamos hook_menu_local_task_alter () para evitar la creación de una nueva ruta de acceso del router" Excepto envolver a los 80 caracteres. Re-laminado contra la cabeza. Cambiado los comentarios en línea para que afirman lo que hacen real: "Añadir enlace de acción de 'some / where / else' en 'otro / página'." El sistema de menú sólo es compatible con las tareas y acciones que tienen la ruta de enrutador tab_parent como su antepasado locales. Usted puede hacer una acción 'admin / content / añadir "local y que se mostrará en' admin / contenido". Pero no se puede hacer una acción local 'nodo / add' y lo mostrará en 'admin / contenido ". Ese es uno de los casos de uso del gancho alter se implementó para. Por lo tanto, este parche simplemente implementa este gancho para eliminar un alter WTF tematización y dar consistencia a la salida. coger Atribución de crédito: captura comentado 3 diciembre 2009 a las 24:26 »Revisado probado por la comunidad Esos comentarios se ven mejor, de nuevo a RTBC. Se seca Atribución de crédito: Dries comentado 3 diciembre 2009 a las 15:45 Se supone que esto para traer de vuelta el enlace "Añadir nuevo bloque" en la página de administración de bloques? Si es así, no creo que se está trabajando. sol de crédito Reconocimiento: sol comentado 3 diciembre 2009 a las 16:09 Se seca Atribución de crédito: Dries comentado 3 diciembre 2009 a las 20:21 Comentado probado por la comunidad Comprometida con CVS HEAD. ¡Gracias! 17 de diciembre 2009 a las 20:30 cerrado automáticamente - Solucionado el problema por 2 semanas sin actividad. c960657 Atribución de crédito: c960657 comentó 20 de mayo 2010 a 20:40 La involuntaria commit mencionado en el # 45 no se ha revertido. Me acaba de presentar # 804844: Revertir la revisión 1.29 en default. settings respecto a esto.