pixelgrade / customify

Intuitive Website Styling integrated into WordPress' Customizer
GNU General Public License v2.0
28 stars 5 forks source link

Update documentation to match new Customizer sections organization - shop themes #140

Closed vladolaru closed 4 years ago

vladolaru commented 6 years ago

O data cu update-ul la 1.7.4 care include reorganizarea sectiunilor din Customizer trebuie sa updatam si documentatia temelor cu noile locatii. De ex. Appearance > Main Content > ... devine Appearance > Theme Options > Main Content > ...

Toate sectiunile introduse de teme se muta in panoul Theme Options.

Propun ca intai sa facem modificarile doar la documentia temelor de pe shop, si apoi la celelalte.

@pixelgrade/heroes @georgeolaru

raduconst commented 6 years ago

@vladolaru Ma pot ocupa eu de modificarile din documentatie. Mai intai insa cred ca trebuie sa dam update la Customify (nu stiu daca implica update si la teme) pentru a aparea aceste schimbari in customizer, nu?

vladolaru commented 6 years ago

@raduconst Nu implica modificari in teme. Doar prin update la Customify.

raduconst commented 6 years ago

@georgeolaru Tocmai ma ocupam de issue-ul asta si, din cautarea ce o faceam prin documentatie, am dat de articolul asta: https://pixelgrade.com/docs/design-and-style/style-changes/changing-colors/ Acum ca aceasta zona de culori e diferita, cred ca acest articol va trebui inlocuit integral cu unul despre Style Manager, nu? In articolul pe care l-am mentionat eu e detaliat cum ar trebui sa aleaga un user culorile si ce sa urmareasca. Acum insa, Style Manager ar trebui sa faca toata treaba pentru el iar articolul Changing Colors nu cred ca isi mai are rostul. Ce parere ai?

georgeolaru commented 6 years ago

@raduconst good catch! Cred ca putem pastra articolul — informatiile generale legate de culori inca mi se par utile (color meaning, contrast).

Poate putem doar inlocui paragraful There are three major areas where you can make color adjustments: Header, Main Content and Grid Blog Items. Just navigate to Appearance → Customize select the desired area where and hit the Colors tab. cu ceva legat de faptul ca optiunile de culori se afla in Style Manager → Colors

raduconst commented 6 years ago

@vladolaru Am facut progrese in ceea ce priveste articolul de aici insa m-am dat un pas in spate si am observat ca asta exista pentru moment: https://pixelgrade.com/wp-admin/post.php?post=13196&action=edit insa este doar pentru Gema, Hive si Patch. As putea sa il adaug si pentru celelalte teme de pe shop insa exista cateva inconsistente la care as avea nevoie de putin ajutor:

  1. La Silk, zonele sunt diferite: https://cl.ly/450b0w3i2s0R (comparativ cu Felt, Julia sau alte teme: https://cl.ly/2s072O0R1q1B);
  2. Prima zona e uneori Site Identity (Julia, Felt, Vasco, Hive, Silk), uneori e Site Title & Logo (Patch, Gema); ma gandesc ca ar trebui sa pastram o consistenta;
  3. Zonele de optiuni cum ar fi Content Options la Julia, Felt, Vasco (https://cl.ly/2x1b232q1k17) sau Theme la Patch (https://cl.ly/2U2J2T3X3S3N) sau Hive Options la Hive (https://cl.ly/1m3g2s2F3c0L) nu ar trebui sa fie incluse in Theme Options din prima zona? Practic tot optiuni de tema sunt si cred ca tot acolo ar trebui sa intre.

Cam asta am gasit eu din sapatul pentru a modifica articolul in documentatie si m-am gandit ca mai intai sa vedem cum structuram mai bine lucrurile decat sa umplu (si sa complic) un articol cu [specific] la tot pasul.

georgeolaru commented 6 years ago

@raduconst intr-adevar nu am luat in considerare ca poate sa existe deja o sectiune "Theme Options" 🤔 Ma gandesc ca ar fi util daca am putea aduna optiunile intr-o sectiune in Theme Options → "General"

vladolaru commented 6 years ago

@raduconst @georgeolaru Stiu de cazul asta si il vom rezolva prin update la tema. Eu cred ca nu are rost sa "stricam" logica din Style Manager cu lucruri care nu tin de el.

Legat de Site Identity vs Site Title & Logo, acestea sunt sectiuni din core-ul WordPress, nu sunt adaugate de noi. S-ar putea la Patch, Gema sa fi modificat noi titlul. Again, fix prin update la tema.

Legat de Content Options, aceea e o sectiune introdusa de Jetpack si nu stiu daca ar trebui sa ne atingem de ea. Acum grupa doar optiunile introduse de tema.

raduconst commented 6 years ago

@vladolaru Am inteles care e treaba cu Content Options si faptul ca va trebui sa dam un update la Silk, Patch si Gema.

Insa zona cu Theme de la Patch, de exemplu, ma gandeam ca poate fi inclusa in Theme Options: https://cl.ly/2M1i0D1L0d0Xhttps://cl.ly/390D2b0b3k2Z de vreme ce optiunile din zona Theme sunt optiuni de tema si ar trebui sa fie in Theme Options (parerea mea). Nu cred ca asta are vreo legatura cu Style Manager, eu ma refer strict la Theme Options si la faptul ca nu prea are sens sa existe doua zone care, in esenta au optiuni pentru acelasi lucru (in cazul de fata, Theme Options si Theme).

vladolaru commented 6 years ago

@raduconst Am introdus in Customify logica de cleanup care incerca sa identifice sectiunile introduse de teme direct (fara logica din Customify) si sa mute toate controalele de acolo intr-o sectiune Theme Options > General. Va fi in update-ul la Customify 1.9.0

vladolaru commented 6 years ago

@raduconst Inchizi tu aici cand e gata totul?

vladolaru commented 5 years ago

@alinclamba Poti prelua tu aici?

alinclamba commented 5 years ago

@vladolaru sigur că da, mersi de heads-up.

georgeolaru commented 4 years ago

@alinclamba Mai e valabil acest issue sau putem sa-l inchidem?

alinclamba commented 4 years ago

Eu am uitat de acest issue, îmi pare rău. @madalingorbanescu pregătește un update pentru Customify și așa a dat peste el. Știu că în ultima perioadă, @andreiungurianu s-a ocupat în mod special de documentație. Andrei, ne poți spune dacă ai pe lista ta și asta?

andreiungurianu commented 4 years ago

Salutare. Eu am făcut săptămâna trecută update pe următoarele articole care se intersectau cu opțiunile descrise mai sus:

Singurele care mai sunt în categoria Design and Style sunt Theme Customizer (valabil pentru Timber), Portfolio Grid (valabil pentru Noah) și Post Titles Styling (valabil pentru Hive). Le pot trece și pe astea pe listă să mă asigur că indicațiile se potrivesc cu locul setărilor.

În rest, e posibil să mai fie articole ce se intersectează cu zone din Theme Options (ex partea de General - ca în cazul indicațiilor legate de Google Maps API), însă pe măsură ce voi ajunge și la articolele respective, mă voi ocupa de ele.

Eu zic că după ce verific articolele menționate de mine, pot închide issue-ul.

@madalingorbanescu și @alinclamba - dacă aveți idee de articole care credeți că au fost afectate de modificările menționate în acest issue, let me know ca să le pun pe to do.

vladolaru commented 4 years ago

Bun asa @andreiungurianu

PS: This is public so we should be using English. But since this is a previously opened issue, I will let it slide :)

andreiungurianu commented 4 years ago

Hello. I managed to update all of these three articles that I mentioned above. It seems like all of them were outdated regarding the location of the settings from the Customizer.

I also managed to edit the video from Portfolio Grid , replacing the settings pannel with the new updated one.