Semantic-Org/Semantic-UI-React

feat(Dropdown): support MenuItem and Button types

Open

#642 geöffnet am 6. Okt. 2016

Auf GitHub ansehen
 (14 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (13.136 Stars) (4.065 Forks)batch import
enhancementgood first issuehelp wanted

Beschreibung

Steps

I'm using a Dropdown as Menu.Item and noticed that the width of the menu item seems off. I checked the Semantic-UI docs (3rd example) and they are using an additional icon class for the menu item if the Dropdown only contains an Icon.

Expected

class="ui dropdown icon item"

Result

class="ui dropdown item"

Workaround

adding className='icon'as additional Dropdownprop

Testcase

http://codepen.io/Chlris-R3/pen/PGQbqZ

  • left: without icon class
  • right: using className='icon' as additional Dropdownprop

Is this a bug / missing feature / working as intended?

Contributor Guide

Tech Stack
reactjavascriptcss
Domain
frontend
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
ReactSemantic UI React componentsJSX
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
The issue requests that when a Dropdown is used as a Menu.Item, the component should automatically add the 'icon' class. Inspect the Dropdown component (likely in src/modules/Dropdown/Dropdown.js) and the Menu.Item component (src/collections/Menu/MenuItem.js) to understand how className is constructed. The fix likely involves checking if the Dropdown contains only an Icon and adding the 'icon' className accordingly, similar to Semantic UI's behavior.