Ajay Pratap

Results 12 comments of Ajay Pratap

> Still seeing the divider mismatch on the latest surge. @kmcfaul @wise-king-sullyman On local changes look fine but seems changes are not reflecting on surge. ![image](https://github.com/patternfly/patternfly-react/assets/11775081/e23b3914-a878-45f7-8552-a67ba5ecd050)

> Still seeing the divider mismatch on the latest surge. Seems changes are not reflecting on surge. Please check it on local. It seems fine on local.

> @ajaypratap003 this demo was recently added to the v6 branch. Perhaps you could cherry pick that commit, or you could make sure they align. @tlabaj Can you please share...

@tlabaj Can you please merge this PR if everything look good.

> Nice work! Looks like you need to add `toolbar` to `.pf-v5-c-masthead__content`. > > @mattnolting Sorry, I don't get this point. Can you please describe bit more

> `nav` goes inside a `toolbar__item` with the class `pf-m-overflow-container` > > **Current** > > > **update** nav is already inside toolbar_item. this is the latest commit https://github.com/patternfly/patternfly-react/pull/10241/commits/525f4a7b7b89695cc40fc1a4d7fac8d79aa6ce00

Thanks you all for reviewing PR. In Github issue, it was about the static pagination. That's why I made it static pagination, not dynamic. Anyway I made it dynamic and...

Hey folks, Can you please merge this PR if everything looks fine

closing this PR as this demo does not match any core demos and It is not needed.

> The core demo is setup so that the drawer is attached to the `` component's main content area. AFAIK the only way to do that on the react side...