Software Público Brasileiro

Reassigned to @mfdeveloper

Pq não utilizar um mock deste serviço nesse teste? Algum motivo em especial? Em linhas gerais, temos mocado todas as dependências nos outros testes, correto?

Talvez seja interessante padronizar uma abordagem geral, para que todos nós saibamos quando (ou não) utilizar mocks para as dependências. Principalmente nesse caso q este serviço utiliza um EventEmitter internamente :)

a idéia é fazer mock das dependências e não do próprio componente "under test". Nesse caso como ele não tem dependências não precisei criar mocks. Apenas utilizei "stubs" onde precisei

Não seria mais interessante passar somente as classes css aq neste atributo e mover o HTML para o template? Me pareceu meio "poluído" este HTML dentro do componente.

OK, Feito

Outros componentes que quiserem utilizar o serviço DesignModeService precisariam criar um get/set dessa forma?

Não. utilizado apenas para bind com o componente visual

Houve modificações nesse arquivo realmente?

Sim...

div ui-toolbar

Este css n deveria ficar no arquivo .sass do component DesignModeTogglerComponent ?

OK

Este css n deveria ficar no arquivo .sass do component DesignModeTogglerComponent?

OK

Essa nova toolbar é especifica do profile ou do contexto geral da aplicação? Ela só será exibida ao acessar profiles (pessoas, comunidades...) ?

A idéia é que possa apresentar botões relativos aos profiles e de contexto geral. No momento existe apenas um botão (toggler) que é comum mas a idéia é ter opções diferenciadas se for profile ou se for environment

Segui o que é feito com o navbar-actions.

Subtemas deveriam sobrescrever os arquivos .scss do tema principal n? Não entendi muito bem como está essa estrutura de modificações de sass em subtemas, mas colocar tdo isso em um único sass me parece "quebrar" a organização do tema principal :(

Quem usa essa interface? Não consegui encontrar o seu uso

Para uso futuro. A idéia é ter uma interface para componentes com controles de permissões na exibição.

Pq está repetido o mesmo nome do atributo na anotação @Input() ? Por padrão ele não utiliza o mesmo nome do atributo?

Algumas sugestões de alteração visual:

O botão de toggle está com uma borda forte quando está em foco (:focus). Talvez igualar o mesmo estilo do :hover já melhore bastante!

toggle-focus

Sugiro remover o gradiente da nova toolbar e ajustar a cor de fundo para um cinza claro e sucinto: #edecec

background-grey

Added 14 new commits:

a8b16135 - Fix application startup process
dc8c43fd - adding KNOWN ISSUES section to README.md
6a66f44b - Merge branch 'master' of softwarepublico.gov.br:noosfero-themes/angular-theme
63a0a102 - Fix application startup process
d00c6e8c - Added export comments button on comment paragraph plugin
ba519042 - Display button to export comments only to users with permission to edit article
36e4d9d9 - Merge branch 'export-comments' into 'master'
f276722d - New npm config 'skin' to allow sass by theme skins
04a99db3 - Merge branch 'theme-skin-yellow'
69ff070c - Component to search for articles in the environment
5bc45a30 - fixed pt translation
575daccf - changed search.component and the search-form.component to fill the query field w…
d0fa7823 - Merge branch 'search' into 'master'
f4964918 - merge with master. fixed toolbar background-color. small fixes on BootstrapSwitcherItem

melhorias e ajustes finos serão feitos em nova issue #95

Status changed to closed

Status changed to reopened

mentioned in commit 9d2ce38269021599b00f6f56c51207a021c2566e

mentioned in commit 9f5c1aa1ff115e8e76be0e77e120d1b726cf5602

mentioned in merge request !42

mentioned in commit 8690849bc505ac8e8581406b4993aebb453aecfb

mentioned in commit f74c41d25288ab4a55b81192240ecfce965ef4ce

	3	+import {DesignModeTogglerComponent} from './designModeToggler.component';
	4	+import {DesignModeService} from './designMode.service';
	5	+
	6	+describe('DesignModeToggler Component', () => {
	7	+ const htmlTemplate: string = '<noosfero-design-toggler></noosfero-design-toggler>';
	8	+
	9	+ let helper: ComponentTestHelper<DesignModeTogglerComponent>;
	10	+ beforeEach(() => {
	11	+ angular.mock.module('templates');
	12	+ angular.mock.module('ngSanitize');
	13	+ angular.mock.module('toggle-switch');
	14	+ });
	15	+
	16	+ let designModeService: DesignModeService;
	17	+ beforeEach((done) => {
	18	+ designModeService = new DesignModeService();
2		Link here Michel Felipe @mfdeveloper 2016-06-14 13:53:14 UTC (Edited 2016-06-14 14:01:39 UTC ) Pq não utilizar um mock deste serviço nesse teste? Algum motivo em especial? Em linhas gerais, temos mocado todas as dependências nos outros testes, correto? Talvez seja interessante padronizar uma abordagem geral, para que todos nós saibamos quando (ou não) utilizar mocks para as dependências. Principalmente nesse caso q este serviço utiliza um `EventEmitter` internamente :) Write Preview Pq não utilizar um mock deste serviço nesse teste? Algum motivo em especial? Em linhas gerais, temos mocado todas as dependências nos outros testes, correto? Talvez seja interessante padronizar uma abordagem geral, para que todos nós saibamos quando (ou não) utilizar mocks para as dependências. Principalmente nesse caso q este serviço utiliza um `EventEmitter` internamente :) Choose File ... File name... Cancel Link here Ábner Oliveira @abner 2016-06-15 19:50:54 UTC a idéia é fazer mock das dependências e não do próprio componente "under test". Nesse caso como ele não tem dependências não precisei criar mocks. Apenas utilizei "stubs" onde precisei Write Preview a idéia é fazer mock das dependências e não do próprio componente "under test". Nesse caso como ele não tem dependências não precisei criar mocks. Apenas utilizei "stubs" onde precisei Choose File ... File name... Cancel

	1	+import {Component, Inject} from 'ng-forward';
	2	+import {DesignModeService} from './designMode.service';
	3	+@Component({
	4	+ selector: 'noosfero-design-toggler',
	5	+ templateUrl: 'app/admin/layout-edit/designModeToggler.html'
	6	+})
	7	+@Inject(DesignModeService)
	8	+export class DesignModeTogglerComponent {
	9	+
	10	+ icon: string = " <i class='glyphicon glyphicon-wrench'></i> ";
2		Link here Michel Felipe @mfdeveloper 2016-06-14 14:03:20 UTC Não seria mais interessante passar somente as classes css aq neste atributo e mover o HTML para o template? Me pareceu meio "poluído" este HTML dentro do componente. Write Preview Não seria mais interessante passar somente as classes css aq neste atributo e mover o HTML para o template? Me pareceu meio "poluído" este HTML dentro do componente. Choose File ... File name... Cancel Link here Ábner Oliveira @abner 2016-06-15 19:48:55 UTC OK, Feito Write Preview OK, Feito Choose File ... File name... Cancel

	2	+import {DesignModeService} from './designMode.service';
	3	+@Component({
	4	+ selector: 'noosfero-design-toggler',
	5	+ templateUrl: 'app/admin/layout-edit/designModeToggler.html'
	6	+})
	7	+@Inject(DesignModeService)
	8	+export class DesignModeTogglerComponent {
	9	+
	10	+ icon: string = " <i class='glyphicon glyphicon-wrench'></i> ";
	11	+
	12	+ constructor(private designModeService: DesignModeService) {
	13	+ }
	14	+
	15	+ private _inDesignMode: boolean = false;
	16	+
	17	+ get inDesignMode(): boolean {
2		Link here Michel Felipe @mfdeveloper 2016-06-14 14:04:22 UTC Outros componentes que quiserem utilizar o serviço `DesignModeService` precisariam criar um get/set dessa forma? Write Preview Outros componentes que quiserem utilizar o serviço `DesignModeService` precisariam criar um get/set dessa forma? Choose File ... File name... Cancel Link here Ábner Oliveira @abner 2016-06-15 19:48:47 UTC Não. utilizado apenas para bind com o componente visual Write Preview Não. utilizado apenas para bind com o componente visual Choose File ... File name... Cancel

35	35	padding: 0 20px 20px 20px;
36	36	}
37	37	}
	38	+
	39	+
	40	+body.noosfero-design-on {
	41	+
	42	+ div.content-wrapper {
	43	+ opacity: 0.5;
	44	+ }
	45	+}
	46	+
	47	+ul.nav > li.toggler-container {
2		Link here Michel Felipe @mfdeveloper 2016-06-14 14:28:19 UTC Este css n deveria ficar no arquivo .sass do component `DesignModeTogglerComponent` ? Write Preview Este css n deveria ficar no arquivo .sass do component `DesignModeTogglerComponent` ? Choose File ... File name... Cancel Link here Ábner Oliveira @abner 2016-06-15 19:44:32 UTC OK Write Preview OK Choose File ... File name... Cancel

289	289	.pace .pace-progress {
290	290	background-color: #fff;
291	291	}
	292	+
	293	+ noosfero-design-toggler .ats-switch .knob i {
2		Link here Michel Felipe @mfdeveloper 2016-06-14 14:28:57 UTC (Edited 2016-06-14 17:36:18 UTC ) Este css n deveria ficar no arquivo .sass do component `DesignModeTogglerComponent`? Write Preview Este css n deveria ficar no arquivo .sass do component `DesignModeTogglerComponent`? Choose File ... File name... Cancel Link here Ábner Oliveira @abner 2016-06-15 19:42:42 UTC OK Write Preview OK Choose File ... File name... Cancel

Software Público Brasileiro

Merge Request #39

Design mode - Added 'inDesign' Toggler

Merged by Ábner Oliveira 2016-06-15 15:49:13 UTC

26	26	<span ng-bind="ctrl.currentUser.person.name"></span> <b class="caret"></b>
27	27	</a>
28	28	<ul class="dropdown-menu" uib-dropdown-menu>
29		- <li>
3		Link here Michel Felipe @mfdeveloper 2016-06-14 14:26:43 UTC Houve modificações nesse arquivo realmente? Write Preview Houve modificações nesse arquivo realmente? Choose File ... File name... Cancel Link here Ábner Oliveira @abner 2016-06-15 19:43:45 UTC Sim... Write Preview Sim... <div ui-view="toolbar"> Choose File ... File name... Cancel Link here Ábner Oliveira @abner 2016-06-15 19:44:15 UTC div ui-toolbar Write Preview div ui-toolbar Choose File ... File name... Cancel

	1	+import {Component, Inject, provide} from "ng-forward";
	2	+import {ProfileService} from "../../lib/ng-noosfero-api/http/profile.service";
	3	+
	4	+@Component({
	5	+ selector: "profile-toolbar",
	6	+ templateUrl: "app/profile/toolbar.html",
	7	+ providers: [
	8	+ provide('profileService', { useClass: ProfileService })
	9	+ ]
	10	+})
	11	+@Inject(ProfileService)
	12	+export class ProfileToolbarComponent {
2		Link here Michel Felipe @mfdeveloper 2016-06-14 14:38:41 UTC Essa nova toolbar é especifica do profile ou do contexto geral da aplicação? Ela só será exibida ao acessar profiles (pessoas, comunidades...) ? Write Preview Essa nova toolbar é especifica do profile ou do contexto geral da aplicação? Ela só será exibida ao acessar profiles (pessoas, comunidades...) ? Choose File ... File name... Cancel Link here Ábner Oliveira @abner 2016-06-15 15:39:01 UTC A idéia é que possa apresentar botões relativos aos profiles e de contexto geral. No momento existe apenas um botão (toggler) que é comum mas a idéia é ter opções diferenciadas se for profile ou se for environment Segui o que é feito com o navbar-actions. Write Preview A idéia é que possa apresentar botões relativos aos profiles e de contexto geral. No momento existe apenas um botão (toggler) que é comum mas a idéia é ter opções diferenciadas se for profile ou se for environment Segui o que é feito com o navbar-actions. Choose File ... File name... Cancel

33	33	background-color: #7E7E7E;
34	34	}
35	35	}
	36	+
	37	+.ats-switch {
1		Link here Michel Felipe @mfdeveloper 2016-06-14 14:47:38 UTC Subtemas deveriam sobrescrever os arquivos .scss do tema principal n? Não entendi muito bem como está essa estrutura de modificações de sass em subtemas, mas colocar tdo isso em um único sass me parece "quebrar" a organização do tema principal :( Write Preview Subtemas deveriam sobrescrever os arquivos .scss do tema principal n? Não entendi muito bem como está essa estrutura de modificações de sass em subtemas, mas colocar tdo isso em um único sass me parece "quebrar" a organização do tema principal :( Choose File ... File name... Cancel

	1	+
	2	+
	3	+export interface IComponentWithPermissions {
2		Link here Michel Felipe @mfdeveloper 2016-06-14 14:49:35 UTC Quem usa essa interface? Não consegui encontrar o seu uso Write Preview Quem usa essa interface? Não consegui encontrar o seu uso Choose File ... File name... Cancel Link here Ábner Oliveira @abner 2016-06-15 19:47:58 UTC Para uso futuro. A idéia é ter uma interface para componentes com controles de permissões na exibição. Write Preview Para uso futuro. A idéia é ter uma interface para componentes com controles de permissões na exibição. Choose File ... File name... Cancel

	9	+ selector: 'noosfero-bootstrap-switcher',
	10	+ template: `
	11	+ <span class="switcher-label" ng-bind="ctrl.label \| translate"></span>
	12	+ <div class="btn-group switcher">
	13	+ <button ng-repeat="option in ctrl.options track by $index"
	14	+ (click)="ctrl.switcherClick(option)"
	15	+ ng-class="ctrl.getCssClassForItem(option)"
	16	+ class="btn btn-xs" ng-bind="option.label \| translate">
	17	+ </button>
	18	+ </div>
	19	+ `,
	20	+ inputs: ['label', 'options', 'defaultOption'],
	21	+ outputs: ['onSwitch']
	22	+})
	23	+export class BootstrapSwitcherComponent {
	24	+ @Input('activeClass') activeClass: string = 'active btn-danger';
1		Link here Michel Felipe @mfdeveloper 2016-06-14 14:51:50 UTC Pq está repetido o mesmo nome do atributo na anotação `@Input()` ? Por padrão ele não utiliza o mesmo nome do atributo? Write Preview Pq está repetido o mesmo nome do atributo na anotação `@Input()` ? Por padrão ele não utiliza o mesmo nome do atributo? Choose File ... File name... Cancel

Merge Request #39

How to merge

Design mode - Added 'inDesign' Toggler

Merged by Ábner Oliveira 2016-06-15 15:49:13 UTC $('.time_ago').timeago().tooltip()

Merged by Ábner Oliveira 2016-06-15 15:49:13 UTC