Skip to content
This repository has been archived by the owner on Apr 1, 2022. It is now read-only.

General Little Improvements #29

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

NicoAntonelli
Copy link
Contributor

@NicoAntonelli NicoAntonelli commented Apr 4, 2021

@NicoAntonelli NicoAntonelli self-assigned this Apr 4, 2021
@NicoAntonelli NicoAntonelli added the enhancement New feature or request label Apr 4, 2021
@@ -3,7 +3,8 @@
<mat-form-field class="search-bar">
<mat-label>Buscar proyectos</mat-label>
<input matInput placeholder="Por nombre de proyecto o por usuario" formControlName="generalSearch" />
<mat-icon matSuffix (click)="onSubmit()">search</mat-icon>
<mat-icon *ngIf="generalSearchValid()" matSuffix>search</mat-icon>
<mat-icon *ngIf="!generalSearchValid()" matSuffix type="button" class="submitIcon" (click)="onSubmit()">check</mat-icon>
Copy link
Contributor

Choose a reason for hiding this comment

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

Puede ser que no se vea clickeable en desktop?

Copy link
Contributor

Choose a reason for hiding this comment

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

Tal vez deberiamos usar mat-button aca tambien en lo posible

@alereca alereca force-pushed the General-little-improvements branch from 5e69e5f to 13dd8ee Compare April 6, 2021 22:58
@alereca alereca force-pushed the General-little-improvements branch from fc267ab to 8b1f63b Compare April 7, 2021 03:21
@alereca alereca force-pushed the General-little-improvements branch from 8b1f63b to 8d3e469 Compare April 7, 2021 03:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants