Mais uma contribuição

Depois da minha primeira contribuição e vasculando um pouco mais o repositório do Settings Gnome, eu, Andrew e Gustavo decidimos fazer um trio para fazermos as próximas issues já que elas pareciam ser um pouco mais complexas!

Então fizemos uma reunião e percebemos que a primeira contribuição que o Andrew fez com o Gustavo, poderia ser aplicado em outros contextos, essa contribuição seria para remover uma linha “invisivel” ao final de uma lista de itens (https://gitlab.gnome.org/GNOME/gnome-control-center/-/issues/1644). Ao invés de resolver de vez essa issue, decidimos criar uma issue e permitir que outros contribuidores possam fazer first touch na aplicação

Network Extra line below the Wi-Fi and VPN connections list - https://gitlab.gnome.org/GNOME/gnome-control-center/-/issues/3461

Segundo MR

No mesmo caminho, como decidimos unir forças, continuamos com outra contribuição, sendo essa de adicionar um toast após a deleção de alguma configuração de qualquer app adicionado ao sistema. O que acontecia antes era a deleção dessa configuração sem nenhum feedback aparente, o que impossibilitava o usuário que refazer a exata ação que ele fez e de ter um feedback visual do que acabou de acontecer.

A primeira coisa que eu fiz para essa contribuição foi analisar o componente Toast que estava sendo usado em outros componentes pelo repositório, eu usei bastante de referência o Toast da deleção de um Background, onde ele usa o component AdwToast e faz um tratamento da quantidade de componentes que estão sendo deletados.

Após isso eu criei minha primeira implementação… o resultado dela foi a criação de um Toast que informava a quantidade de deleção para cada tipo de arquivo, então recebi meu primeiro feedback no qual do Felipe Borges com várias coisas para resolver no código e além disso com uma sugestão de não mostrar a quantidade e sim só deletar e se a pessoa deletar outra configuração a ultima desaparece e mostra a nova:

Image

Eu fiz essa alteração meio que contra gosto, porque eu gosto da ideia de contar os itens e ai veio o Matthijs Velsink, outro mantenedor e sugeriu que tirassemos a separação dos tipos e colocassemos apenas type e a quantidade de itens removidos, o que para mim foi ótimo e amei a sugestão.

Image

Então fui eu lá implementar novamente a segunda sugestão dos mantenedores e criar um novo commit, para que se eles decidirem por ir no caminho da primeira sugestão eu possa fazer um rollback e recuperar o qu eu tinha feito.

Image

Segue o link do MR:

https://gitlab.gnome.org/GNOME/gnome-control-center/-/merge_requests/3156

Acredito que como esse MR envolve muitas alterações ele possa demorar um pouco para ser aprovado, mas de qualquer forma eu achei a respostas dos mantenedores rápida e estou acompanhando as alterações para que eu consiga responde-los o mais rapido possivel.