Vyriešené... ďakujem

načo vymýšlať koleso, pravda? Ale každopádne, ak chceš používať svoj modal a nie niektorý z tých desiatok čo sú bežne používané, tak len malé výhrady k tomuto:

overlay by bolo fajn vyčleniť z toho modalu, lebo takto sa budú zrátavať. Pri niekoľkých modaloch máš už čierne pozadie a vôbec to neplní funkciu na ktorú to je určené. Mal by si zabezpečiť, aby bol overlay vždy len jeden (napríklad javascriptom, aj tak ho tam budeš potrebovať na zatváranie/otváranie/presúvanie toho okna ak to má byť plnohodnotný modal). Ak nechceš používať javascript, vieš to ošetriť aj selectorom v css aby sa tá definícia týkala len prvého

tá mediaquery by mala byť na konci, vyčlenená z triedy a v nej prepísať jednotlivé triedy. Takto, ak budeš mať dlhé css to budeš mať roztrúsené po celom súbore a budeš to mať organizované podla tried, a nie podľa rozlíšenia ako keby boli všetky media query na konci, tam si nájdeš aké rozlíšenie a v ňom si nájdeš novú definíciu triedy. Ono vlastne celý ten zápis je divný, že v triede je ďalšia trieda. Je to nejaká featura toho preprocesoru? Treba ho tam vôbec, kvôli pár riadkom csska?

neviem či je dobrý nápad pracovať s fixnou šírkou a výškou toho modalu. Nebolo by lepšie, keby sa prispôsobil obsahu? Takto tam musíš potom čarovať s premennými (čo je pre css dosť časovo náročná operácia) a zbytočne pre jedno písmenko v modale zakryješ 80% obrazovky.

a dizajnovo by som tomu pridal nejaký box-shadow aby bolo vidno že je to niečo "nad" a nie len pacnutý div. Nech to má nejakú hlbku. Taktiež trošku zaobliť rohy... toto mi príde také retro z 90-tych rokov, ale v takejto forme v súčasnosti nepoužitelné. A neviem na čo to plánuješ používať, ale dobre by bolo pridať aj nejakú lištu kde bude minimálne nadpis čoho sa ten modal týka