Scouterna / Scouterna-WPTheme

3 stars 0 forks source link

Search button text #10

Closed mrManner closed 11 years ago

mrManner commented 11 years ago

No biggie, but ehrm…

http://d.pr/i/wsnx

mrManner commented 11 years ago

Oh, man kanske ska köra svenska wp ändå... #tankspridd

Stänger.

sakjur commented 11 years ago

Still a bug, the theme should be fully functional in both sv and en

frozzare commented 11 years ago

Bilden för sökknappen är bara gjord för att passa ordet "Sök", så man behöver fixa bilden så den går att repetera och fixa en css grej så kommer det se bra ut även för ordet "Search".

sakjur commented 11 years ago

men vad tusan. segt. det måste fungera med przeszukiwanie och αναζήτηση också

mrManner commented 11 years ago

Vore enklare att byta ut texten mot ett förstoringsglas... Inte riktigt lika bra funktionalitet, men ser snyggare ut.

sakjur commented 11 years ago

Jag vet inte. Explicit är bättre än implicit. Använder folk förstoringsglas för sökrutor i Uganda?

frozzare commented 11 years ago

Tycker nog inte att man ska byta ut texten till ett förstoringsglas utan köra på som de är idag men att fixa så man kan ha längre ord där. Annars börjar man frångå orginal designen mer och mer när man byter ut grejer och de är väl inte tanken? Tanken är väl att de ska vara ett fristående tema med fixar för grejer man upptäcker och fixar för buggar, de ska ändå likna de temat man får i Scouternas WordPress system.

sakjur commented 11 years ago

Absolut - som jag ser det så är det dock ingen strävan efter att få samma feature-set som Scouterna.se, utan ett fungerande tema som stilistiskt passar in i webbplattformen. Min syn på det hela är att det ska utvecklas mot stabilt tema som har möjligheterna att anpassas, är väldokumenterat och har innovativa funktioner som underlättar för admins.

Och såklart att temat inte gör någonting mer än är ett tema.

frozzare commented 11 years ago

Löst det med att göra bilden bredare och fixa lite css. https://github.com/frozzare/Scouterna-WPTheme/commit/c576780e65d4edaf857a797f37af8cf7049e9539 Dock tänkte jag inte på att göra branch av issue #8 så det blir knas om jag gör en pull reuqest nu förens min tidigare pull request är mergad.

sakjur commented 11 years ago

@frozzare Okej, mergade din förra, känns lite riskabelt utan hunnit testa, men wtf ;) :bowling:

sakjur commented 11 years ago

Fixed